-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
hack/doc: Autogen CLI Documentation #2099
hack/doc: Autogen CLI Documentation #2099
Conversation
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.
it seems to build the docs on my local dev box fine. I evaluate the content but see that it
was generated. I'm assuming the generated docs will be linked from some new document
in a separate PR?
Makefile
Outdated
@@ -56,6 +56,9 @@ tidy: ## Update dependencies | |||
clean: ## Clean up the build artifacts | |||
$(Q)rm -rf build | |||
|
|||
doc: ## Generate CLI Documentation |
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.
[Optional] Could this be cli-doc
?
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.
agree, done.
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.
A few formatting nits, otherwise
/lgtm
Co-Authored-By: Eric Stroczynski <[email protected]>
Co-Authored-By: Eric Stroczynski <[email protected]>
Co-Authored-By: Eric Stroczynski <[email protected]>
… into autogen-cli-docs
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.
/lgtm
Description of the change:
Adds
make doc
command to generate CLI documentationMotivation for the change:
Reduce developer friction when making changes to the CLI
Includes sample generated documentation