Skip to content

Conversation

@epage
Copy link
Contributor

@epage epage commented Oct 23, 2025

rustup check was using console::style which will do its own "can I
style" checks, overriding RUSTUP_TERM_COLOR.

See https://docs.rs/console/0.16.1/src/console/utils.rs.html#696-702

epage added 2 commits October 23, 2025 10:53
`rustup check` was using `console::style` which will do its own "can I
style" checks, overriding `RUSTUP_TERM_COLOR`.

See https://docs.rs/console/0.16.1/src/console/utils.rs.html#696-702
@epage epage changed the title fix(check): Subject check to RUSTUP_TERM_COLOR fix(check): Respect RUSTUP_TERM_COLOR in rustup check Oct 24, 2025
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.

It turns out I have somehow missed this point from #4388... Many thanks for fixing that!

@rami3l rami3l added this pull request to the merge queue Oct 24, 2025
Merged via the queue into rust-lang:main with commit 378655c Oct 24, 2025
29 checks passed
@epage epage deleted the console branch October 24, 2025 11:47
epage added a commit to epage/rustup that referenced this pull request Oct 24, 2025
With `rustup check`s colors being testable (rust-lang#4561),
this updates it to match Cargo's colors
epage added a commit to epage/rustup that referenced this pull request Oct 24, 2025
With `rustup check`s colors being testable (rust-lang#4561),
this updates it to match Cargo's colors
epage added a commit to epage/rustup that referenced this pull request Oct 24, 2025
With `rustup check`s colors being testable (rust-lang#4561),
this updates it to match Cargo's colors
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.

2 participants