-
Notifications
You must be signed in to change notification settings - Fork 136
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 a new CLI command: datum format #1570
Add a new CLI command: datum format #1570
Conversation
Signed-off-by: Kim, Vinnam <[email protected]>
Signed-off-by: Kim, Vinnam <[email protected]>
9d8aaa5
to
69acd17
Compare
Signed-off-by: Kim, Vinnam <[email protected]>
Signed-off-by: Kim, Vinnam <[email protected]>
Signed-off-by: Kim, Vinnam <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #1570 +/- ##
===========================================
- Coverage 81.03% 81.03% -0.01%
===========================================
Files 276 278 +2
Lines 32430 32468 +38
Branches 6595 6599 +4
===========================================
+ Hits 26280 26309 +29
- Misses 4696 4705 +9
Partials 1454 1454
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ 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.
Great work! Many thanks.
Summary
datum format
. It displays a list of data format names supported by Datumaro. It can be useful for quick reference of data format name used for other CLI command such asdatum convert -if <data-format> -f <data-format>
.How to test
Added unit tests as well.
Checklist
License
Feel free to contact the maintainers if that's a concern.