Skip to content

Automated nightly rustfmt (2026-09-27) - #299

Open
github-actions[bot] wants to merge 3 commits into
mainfrom
create-pull-request/patch
Open

github-actions[bot] wants to merge 3 commits into
mainfrom
create-pull-request/patch

Conversation

@github-actions

Copy link
Copy Markdown

Automated nightly rustfmt changes by create-pull-request GitHub action

@ldk-reviews-bot

ldk-reviews-bot commented Sep 27, 2026 •

Copy link
Copy Markdown

I've assigned @benthecarman as a reviewer!
I'll wait for their review and will help manage the review process.
Once they submit their review, I'll check if a second reviewer would be helpful.

Nightly rustfmt wrapped these operator chains differently than stable,
which CI uses for its format check. Restore the stable layout so the
check passes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Comment thread .github/workflows/cron-weekly-rustfmt.yml Fixed
Comment thread .github/workflows/cron-weekly-rustfmt.yml Fixed
Comment thread .github/workflows/cron-weekly-rustfmt.yml Fixed
The weekly cron formats with nightly rustfmt, but CI checks formatting
with stable. The two can disagree on layout (e.g. how binary operator
chains are wrapped), so the bot's PRs fail the stable fmt check and
check-protos.

Run stable rustfmt after nightly so the nightly-only options from
rustfmt.toml (import granularity and grouping) are still applied, but
the final layout always matches what CI expects.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@benthecarman
benthecarman force-pushed the create-pull-request/patch branch from 27bbe56 to 606a0cf Compare September 29, 2026 03:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants