Skip to content

style: run rustfmt with nightly toolchain in ci/style.py - #5340

Merged
tgross35 merged 1 commit into
rust-lang:mainfrom
ssenthilnathan3:style/nightly-rustfmt
Aug 2, 2026
Merged

style: run rustfmt with nightly toolchain in ci/style.py#5340
tgross35 merged 1 commit into
rust-lang:mainfrom
ssenthilnathan3:style/nightly-rustfmt

Conversation

@ssenthilnathan3

@ssenthilnathan3 ssenthilnathan3 commented Jul 30, 2026

Copy link
Copy Markdown

Updates the ci/style.py script to automatically invoke rustfmt and
cargo fmt with a nightly toolchain via rustup, so contributors don't need to
figure this out manually.

Closes #5324

@rustbot rustbot added A-CI Area: CI-related items S-waiting-on-review labels Jul 30, 2026
Comment thread ci/style.py Outdated
# if `CI` is set, do a check rather than overwriting
check_only = os.getenv("CI") is not None
run(["rustfmt", "-V"])
run(["rustup", "run", "nightly", "rustfmt", "-V"])

@tgross35 tgross35 Jul 30, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Checking locally, it seems like rustfmt +nightly should work

View changes since the review

@ssenthilnathan3
ssenthilnathan3 force-pushed the style/nightly-rustfmt branch from ec95343 to d91a5d6 Compare July 31, 2026 04:36
@tgross35 tgross35 added the stable-nominated This PR should be considered for cherry-pick to libc's stable release branch label Jul 31, 2026

@tgross35 tgross35 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@tgross35
tgross35 enabled auto-merge July 31, 2026 04:38
auto-merge was automatically disabled July 31, 2026 04:38

Head branch was pushed to by a user without write access

@ssenthilnathan3
ssenthilnathan3 force-pushed the style/nightly-rustfmt branch 2 times, most recently from 9e171ac to 3534200 Compare July 31, 2026 04:38
@xtqqczze

This comment was marked as outdated.

@tgross35
tgross35 enabled auto-merge August 2, 2026 19:30
@tgross35
tgross35 force-pushed the style/nightly-rustfmt branch from 3534200 to cb1954d Compare August 2, 2026 19:30
@rustbot

rustbot commented Aug 2, 2026

Copy link
Copy Markdown
Collaborator

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@tgross35
tgross35 added this pull request to the merge queue Aug 2, 2026
Merged via the queue into rust-lang:main with commit 866b642 Aug 2, 2026
57 checks passed
@tgross35 tgross35 mentioned this pull request Aug 31, 2026
@tgross35 tgross35 added stable-applied This PR has been cherry-picked to libc's stable release branch and removed stable-nominated This PR should be considered for cherry-pick to libc's stable release branch labels Aug 31, 2026
tgross35 pushed a commit to dybucc/libc that referenced this pull request Aug 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-CI Area: CI-related items stable-applied This PR has been cherry-picked to libc's stable release branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ci/style.py only works on nightly

5 participants