Skip to content
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

docs: add cli docs #577

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

docs: add cli docs #577

wants to merge 2 commits into from

Conversation

yathomasi
Copy link
Contributor

From comment

How do we go about docs for these commands? Let's please review and created tickets if needed.

Do we mean sth like this ?

Screenshot 2024-11-08 at 12 36 38

Putting as draft as we need to include all the commands before we merge.

Copy link

cloudflare-workers-and-pages bot commented Nov 8, 2024

Deploying datachain-documentation with  Cloudflare Pages  Cloudflare Pages

Latest commit: fb867df
Status: ✅  Deploy successful!
Preview URL: https://fab3bc71.datachain-documentation.pages.dev
Branch Preview URL: https://add-cli-docs.datachain-documentation.pages.dev

View logs

@amritghimire amritghimire linked an issue Nov 8, 2024 that may be closed by this pull request
Copy link

codecov bot commented Nov 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.83%. Comparing base (e455180) to head (fb867df).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #577   +/-   ##
=======================================
  Coverage   87.83%   87.83%           
=======================================
  Files         100      100           
  Lines        9993     9993           
  Branches     1356     1356           
=======================================
  Hits         8777     8777           
  Misses        873      873           
  Partials      343      343           
Flag Coverage Δ
datachain 87.77% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@shcheklein
Copy link
Member

Do we mean sth like this ?

No. It looks like Python API docs to me. It should be more like what we have in dvc.org for CLI. I'm not sure if there are good example for this engine.

@yathomasi
Copy link
Contributor Author

I see mkdocs support for cli autodocs with mkdocs-click extension i.e for cli using click. But, for us, I see we use argparse.

@iterative/datachain I see we lack proper restructuring for cli commands. So, could you please check the feasibility of using click for proper structure? Also, we can generate the docs using mkdocs-click.

If we don't want to take that road, we should basically follow the same procedure as we do for dvc.org i.e write them manually for all the commands.

@shcheklein
Copy link
Member

So, could you please check the feasibility of using click for proper structure

tbh, I don't like click. To my mind it doesn't provide enough value for an extra dependency, standard lib gives 95% of what we need. Also, click is breaking some basic (to my mind) things. E.g. there is no way you could append -v at the end of the command to debug it (you have to put in the right position, somewhere in between all args).

If we don't want to take that road, we should basically follow the same procedure as we do for dvc.org i.e write them manually for all the commands.

@skshetry what is your take on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add documentation for CLI commands
2 participants