-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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(cli): clarify stop
v. terminate
with Long
descriptions
#11626
docs(cli): clarify stop
v. terminate
with Long
descriptions
#11626
Conversation
- this is one of the most frequently asked questions by users and so could definitely use some clarification - add a `Long` description to the CLI commands that details how `stop` and `terminate` work in more detail - via `make codegen`, this resulted in a `### Synopsis` section in the generated docs as well, which is also helpful Signed-off-by: Anton Gilgur <agilgur5@gmail.com>
Huh, lint timed out. Haven't seen that happen before. Will need to retry that but may need to bump the timeout. Seems like a one-off right now though |
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.
smart to clarify these! thank you
stop
v. terminate
with Long
descriptionsstop
v. terminate
with Long
descriptions
Please fix lint |
Per my comment above, it timed out and needs a retry |
Looks like this is good to merge now, CI is green after retry |
…1626) Signed-off-by: Anton Gilgur <agilgur5@gmail.com>
…goproj#11626) Signed-off-by: Anton Gilgur <agilgur5@gmail.com> Signed-off-by: Dillen Padhiar <dillen_padhiar@intuit.com>
Motivation
Similar to #11625, this is another one of the most frequently asked questions by users and so could definitely use some clarification
Modifications
retry
andresubmit
#11625, add aLong
description to the CLI commands that details howstop
andterminate
work in more detailmake codegen
, this resulted in a### Synopsis
section in the generated docs as well, which is also helpfulDescriptions loosely follow the official issue response in #4454 (comment).
This is effectively a follow-up to #4653
Verification
make codegen
passesNotes for Reviewers
I wrote this as a "fix" commit so that it can be added as a patch to the CLI (and make it into any cherry-picks), but could make it a "docs" commit instead.
Feel free to add more modifications to the long descriptions!
Future Work
I'd like to add these descriptions to the API docs as well, but that seems to be a larger effort as the API docs have no descriptions right now 😕 I'm also not sure where codegen tries to pull the descriptions from, as there are GoDoc descriptions, but perhaps not in the right places?