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
Improve help text #6392
Improve help text #6392
Conversation
@@ -22,13 +22,22 @@ import ( | |||
) | |||
|
|||
var usageListUsersSummary = ` | |||
Lists Juju users allowed to connect to a controller.`[1:] | |||
Lists Juju users allowed to connect to a controller or model. | |||
When used without a model name argument, users relevant to a controller are printed. |
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.
Aren't these supposed to be a single line?
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.
fixed
4cc18c5
to
91e8b6f
Compare
|
||
var usageListUsersDetails = ` | ||
By default, the tabular format is used. | ||
When used without a model name argument, users relevant to a controller are printed. | ||
When used with a model name, users relevant to the specified model are printed.By default, the tabular format is used. |
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.
Line wrapping?
91e8b6f
to
c046849
Compare
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
Fixes: https://bugs.launchpad.net/juju/+bug/1630863
Fixes: https://bugs.launchpad.net/juju/+bug/1631199
Improve help text for list-users command.
Fix help text in upgrade-juju command.