Skip to content

docs(cli): add auto-generated CLI reference #1209

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
Oct 12, 2020

Conversation

nejch
Copy link
Member

@nejch nejch commented Oct 11, 2020

This adds a generated page similar to the API reference, including all the possible commands. It's a long list, but better than nothing. 😁 I wasn't able to group them into sections due to the way they are currently dynamically generated, but maybe there's a way.

I've noticed a lot of people still use the CLI even though it has some issues. I'm hoping this can help people/prevent opening issues that are simply due to missing docs (recent example: #1197), or at least we can more quickly close those by providing the link rather than providing examples every time.

Also means no more manual examples for the CLI are needed, hopefully.

Click also has a drop-in replacement for the sphinx extension used for this PR, so switching at a later point should be easy.

Closes #1197.

@nejch nejch marked this pull request as ready for review October 11, 2020 14:01
@nejch nejch requested a review from max-wittig October 11, 2020 14:01
@codecov-io
Copy link

Codecov Report

Merging #1209 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1209   +/-   ##
=======================================
  Coverage   77.56%   77.56%           
=======================================
  Files          11       11           
  Lines        2826     2826           
=======================================
  Hits         2192     2192           
  Misses        634      634           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 68a4162...6c21fc8. Read the comment docs.

@max-wittig max-wittig merged commit 9054a3b into master Oct 12, 2020
@max-wittig
Copy link
Member

Very nice!

@max-wittig max-wittig deleted the docs/cli-reference-page branch October 12, 2020 15:24
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.

Trigger export using CLI
3 participants