-
Notifications
You must be signed in to change notification settings - Fork 892
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
Implement verbose option for "rustup toolchain list" #1988
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.
Mostly good, I'd like to see the use of clap
not leaked into common.rs
and the tests need a little more thought, but otherwise this is very close to mergeable.
Thank you very much for your contribution and I hope you have time to act on this feedback and provide an updated commit.
Please note that we don't like fixup commits in our history if we can avoid them; so either squash your updates into this commit, or be prepared to be asked to do so once we're happy the content is mergeable.
D.
Thank you for the feedback, @kinnison. Just a quick note to acknowledge the feedback. Will definitely find time to edit to update the PR. |
763dc73
to
37bbba8
Compare
Greeting @kinnison. Per suggestions, I've pushed the updates in a squashed commit to:
PTAL? Thanks. |
6bd2ffd
to
d01c9fb
Compare
d01c9fb
to
e92ca07
Compare
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.
One tiny typo, I can fix it if you don't want to.
Update verbose option for toolchain list Correct toolchain list tests w. verbose option Update syntax based on clippy Simplify toolchain_list fn in rustup_mode.rs Correct typo in list subcommand
e92ca07
to
5406e6f
Compare
Thanks for catching the typo, @kinnison. Updated. |
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.
👍
This PR resolves #1479.
local test
→ rustup toolchain list -h rustup-toolchain-list List installed toolchains USAGE: rustup toolchain list [FLAGS] FLAGS: -h, --help Prints help information -v, --verbose Enable verbose output with toolchain infomation