Skip to content

internal: Skip rustfmt test if nightly toolchain is missing#21681

Merged
lnicola merged 1 commit intorust-lang:masterfrom
Wilfred:skip_test_on_nightly
Feb 24, 2026
Merged

internal: Skip rustfmt test if nightly toolchain is missing#21681
lnicola merged 1 commit intorust-lang:masterfrom
Wilfred:skip_test_on_nightly

Conversation

@Wilfred
Copy link
Copy Markdown
Contributor

@Wilfred Wilfred commented Feb 19, 2026

Currently the rustfmt slow test fails if you don't have a nightly toolchain installed. We install a nightly toolchain on CI, but it's a little confusing that tests can fail on a fresh checkout on a new laptop.

Instead, skip this test if there's no nightly toolchain.

AI disclosure: Written with a little help from Claude.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 19, 2026
@Wilfred Wilfred force-pushed the skip_test_on_nightly branch 2 times, most recently from 8995040 to c199318 Compare February 24, 2026 19:25
@Wilfred
Copy link
Copy Markdown
Contributor Author

Wilfred commented Feb 24, 2026

Argh, cargo clippy doesn't catch issues in the workspace. Hopefully CI should be green now.

Currently the rustfmt slow test fails if you don't have a nightly
toolchain installed. We install a nightly toolchain on CI, but it's a
little confusing that tests can fail on a fresh checkout on a new
laptop.

Instead, skip this test if there's no nightly toolchain.
@Wilfred Wilfred force-pushed the skip_test_on_nightly branch from c199318 to 5752722 Compare February 24, 2026 19:26
@lnicola
Copy link
Copy Markdown
Member

lnicola commented Feb 24, 2026

This makes sense, but I don't understand why the test is green for me without an installed nightly.

EDIT: because RUN_SLOW_TESTS, not SKIP_SLOW_TESTS 😅

@lnicola lnicola added this pull request to the merge queue Feb 24, 2026
Merged via the queue into rust-lang:master with commit ff041aa Feb 24, 2026
16 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 24, 2026
@Wilfred Wilfred deleted the skip_test_on_nightly branch February 24, 2026 22:37
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