-
Notifications
You must be signed in to change notification settings - Fork 55
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
Add csv format to table views #955
Conversation
Adds csv as a general format cli wide Uses csv flag for user search
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #955 +/- ##
==========================================
+ Coverage 72.23% 72.35% +0.12%
==========================================
Files 93 93
Lines 12656 12748 +92
==========================================
+ Hits 9142 9224 +82
- Misses 2969 2976 +7
- Partials 545 548 +3 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the idea of adding a --csv
flag; it was always something we considered during the v1.0.0 initiative. Though I usually like small PRs, I think we should apply this to all commands or at least test against all commands to make sure that the result writer works for all command ouputs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Solid 👍
🔧 Changes
📚 References
--csv
flag toauth0 users search
🔬 Testing
Manually verify that you get a csv back of your search results via
auth0 users search --csv
📝 Checklist