Skip to content

Conversation

@nozwock
Copy link

@nozwock nozwock commented Oct 10, 2025

Closes #4505. Adding a flag to don't try to update already installed toolchains.

Copy link
Contributor

@djc djc left a comment

Choose a reason for hiding this comment

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

I think we'd want some tests for this.

@nozwock

This comment was marked as resolved.

@nozwock nozwock force-pushed the add-install-no-update branch from 72a95f6 to 4192bf4 Compare October 11, 2025 04:45
@djc
Copy link
Contributor

djc commented Oct 13, 2025

Need to update the cli_ui tests. The test failures should give some hints on how to do this efficiently.

@nozwock nozwock force-pushed the add-install-no-update branch 2 times, most recently from 6234419 to c7c20dc Compare November 3, 2025 20:50
@nozwock
Copy link
Author

nozwock commented Nov 3, 2025

Need to update the cli_ui tests. The test failures should give some hints on how to do this efficiently.

Oh I see, I forgot to run all the tests before, only ran cli_rustup. Sorry it's so late, I kinda forgot about the pr.

I just regenerated the help output svg, not sure if something else needs to be done.

@djc
Copy link
Contributor

djc commented Nov 4, 2025

@nozwock do you mind also squashing the test output change into the first commit? Would be nice to have each commit in our branch history pass tests (as far as possible).

@nozwock nozwock force-pushed the add-install-no-update branch from c7c20dc to f09795e Compare November 4, 2025 10:44
@nozwock
Copy link
Author

nozwock commented Nov 4, 2025

@djc I see, didn't think of that. Just did it.

@djc
Copy link
Contributor

djc commented Nov 4, 2025

@rami3l want to take a look?

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.

Add rustup ensure which doesn't check for updates

2 participants