Skip to content

Conversation

@appleGun22
Copy link
Contributor

In continuation to #4694.

Copy link
Member

@rami3l rami3l left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update the test snapshots at the same time with SNAPSHOTS=overwrite so that the CI turns green again. Many thanks for your help!

@appleGun22 appleGun22 requested a review from rami3l February 1, 2026 00:22
Copy link
Member

@rami3l rami3l left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@appleGun22 LGTM!

Please change the commit message and the PR title to better reflect what this patch is doing: clearly it's not about adding --help since it's already been implemented. Rather, you are adding a section about exit codes to it.

Apart from that, just squash all commits into one and we are good to go.

Many thanks again :)

rami3l
rami3l previously requested changes Feb 2, 2026
@rami3l rami3l changed the title Add a --help option for rustup check command feat(cli/rustup-mode): add "Exit status" section to rustup check --help Feb 2, 2026
@rami3l rami3l dismissed their stale review February 2, 2026 18:03

Modified.

@rami3l rami3l requested a review from djc February 2, 2026 18:03
@appleGun22
Copy link
Contributor Author

@rami3l I see that you already squashed them together, but still thank you for the guidance, it was my first PR to rustup :)

@djc djc added this pull request to the merge queue Feb 3, 2026
Merged via the queue into rust-lang:main with commit ac5a50f Feb 3, 2026
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants