Skip to content

Conversation

@joshtriplett
Copy link
Member

@joshtriplett joshtriplett commented Nov 7, 2025

In some cases we have the same subcommand in multiple places, but it accepts different aliases in each place. For instance, you can write either rustup toolchain uninstall or rustup toolchain rm, but you can only write rustup uninstall, not rustup rm.

Add aliases for consistency:

  • rustup add for rustup install, for consistency with rustup toolchain install.
  • rustup rm (or "remove" or "delete" or "del") for rustup uninstall, for consistency with rustup toolchain uninstall.
  • rustup self up for rustup self update, for consistency with other update commands.
  • rustup toolchain up for rustup toolchain update, for consistency with other update commands.

I've also added rustup status as an alias for rustup show, for consistency with git status. 😄

In some cases we have the same subcommand in multiple places, but it
accepts different aliases in each place. For instance, you can write
either `rustup toolchain uninstall` or `rustup toolchain rm`, but you
can only write `rustup uninstall`, not `rustup rm`.

Add aliases for consistency:
- `rustup add` for `rustup install`, for consistency with
  `rustup toolchain install`.
- `rustup rm` (or "remove" or "delete" or "del") for `rustup uninstall`,
  for consistency with `rustup toolchain uninstall`.
- `rustup self up` for `rustup self update`, for consistency with other
  `update` commands.
- `rustup toolchain up` for `rustup toolchain update`, for consistency
  with other `update` commands.
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.

I like it, thanks!

@djc @ChrisDenton Does the show -> status alias seem okay to you as well?

@rami3l rami3l requested review from ChrisDenton and djc November 7, 2025 06:47
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.

@joshtriplett Given the overall feedback from the team, I think maybe we should drop the rustup status part, but other parts of this PR should be fine.

@joshtriplett
Copy link
Member Author

No objection, that part was a convenience anyway. Will rebase that commit away.

@djc djc added this pull request to the merge queue Nov 10, 2025
Merged via the queue into rust-lang:main with commit e49e655 Nov 10, 2025
56 of 58 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.

4 participants