-
Notifications
You must be signed in to change notification settings - Fork 100
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
Review help/usage for cli commands #802
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #802 +/- ##
==========================================
- Coverage 87.61% 87.51% -0.11%
==========================================
Files 128 128
Lines 11324 11326 +2
Branches 1530 1533 +3
==========================================
- Hits 9922 9912 -10
- Misses 1020 1028 +8
- Partials 382 386 +4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@amritghimire thanks! can you please put into description the end result (dump from CLI) please? |
Deploying datachain-documentation with Cloudflare Pages
|
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.
👍
@amritghimire thanks, I've started the document - https://docs.google.com/document/d/1_QeMQ1NsguHSRSyJpF2n-1s57SHSuiyGzl10q9d1UaE/edit?tab=t.0 . Let's iterate there please. |
The pattern followed is: - Descriptions: Complete sentences with periods - Help messages: Concise phrases without periods - Consistent terminology ("Iterative Studio") - Clear, standardized format for similar arguments
c05200d
to
90c54fe
Compare
Updated the PR per the document |
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.
Let's please update the google doc with the new set of outputs, merge this, and do the next iteration. Thanks for driving this @amritghimire !
The pattern followed is:
Final output:
Closes #773