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: update external command documentation #10333
Conversation
Review period will end on 2021-01-18 at 02:52:18 UTC. |
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 again 👏🏻
Co-authored-by: Eric Knibbe <enk3@outlook.com>
Good suggestions, thanks, @EricFromCanada! |
Review period ended. |
Putting this on hold for now until #10342 is merged so that those changes can be reflected here. |
Okay, now that #10342 has been merged, I've updated this PR to include information about auto-generated usage strings. I'll leave this open for now for any last feedback and I'll plan on merging sometime tomorrow. |
LGTM 👍🏻 |
brew style
with your changes locally?brew typecheck
with your changes locally?brew tests
with your changes locally?brew man
locally and committed any changes?Follow up to #10288
This PR updates the documentation around external commands and how to use the new
named_args
method.My ultimate goal is to have completions be automatically generated based on
named_args
and to have an e.g.brew generate-completions
command that will generate completions automatically for a tap. Given this, let me know if it's better for this PR to wait until that's ready (in order to document both at the same time).