Skip to content

feat(cmd): add support for CSV input in apply-users command #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 1, 2025
Merged

Conversation

k1LoW
Copy link
Owner

@k1LoW k1LoW commented Apr 1, 2025

This pull request introduces support for importing users from a CSV file in addition to the existing JSONL format. The changes include updates to the documentation and modifications to the command-line tool to handle the new CSV format.

Documentation updates:

  • README.md: Added a new section describing the CSV format for the users file and how to specify the columns using the --columns flag. [1] [2]

Code updates:

  • cmd/applyUsers.go: Introduced a new variable cols to store the column definitions for the CSV format.
  • cmd/applyUsers.go: Updated the applyUsersCmd command to parse users from a CSV file if the --columns flag is provided.
  • cmd/applyUsers.go: Added a new flag --columns to define the columns for the CSV format.

@k1LoW k1LoW added enhancement New feature or request minor labels Apr 1, 2025
@k1LoW k1LoW self-assigned this Apr 1, 2025
Copy link
Contributor

github-actions bot commented Apr 1, 2025

Code Metrics Report

main (bdf563f) #6 (7120add) +/-
Coverage 0.0% 0.0% 0.0%
Code to Test Ratio 1:0.0 1:0.0 0.0
Test Execution Time 4s 7s +3s
Details
  |                     | main (bdf563f) | #6 (7120add) | +/-  |
  |---------------------|----------------|--------------|------|
  | Coverage            |           0.0% |         0.0% | 0.0% |
  |   Files             |              5 |            5 |    0 |
  |   Lines             |            260 |          272 |  +12 |
  |   Covered           |              0 |            0 |    0 |
  | Code to Test Ratio  |          1:0.0 |        1:0.0 |  0.0 |
  |   Code              |            542 |          564 |  +22 |
  |   Test              |              0 |            0 |    0 |
- | Test Execution Time |             4s |           7s |  +3s |

Code coverage of files in pull request scope (0.0% → 0.0%)

Files Coverage +/-
cmd/applyUsers.go 0.0% 0.0%

Reported by octocov

@k1LoW k1LoW merged commit 56cc073 into main Apr 1, 2025
1 check passed
@k1LoW k1LoW deleted the csv branch April 1, 2025 01:30
@github-actions github-actions bot mentioned this pull request Apr 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request minor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant