style: cargo fmt, so CI stops failing on every PR - #9
Draft
OxToF wants to merge 1 commit into
Draft
Conversation
`cargo fmt --all --check` has been red on main, which makes the Anchor Build & Lint job fail on any branch that touches nothing Rust at all — two PRs merged today both showed a red check they had no part in. A lint that is always red stops being read. Formatting only, no semantic change: `cargo check` clean. One comment moved rather than reformatted. rustfmt treated the FOUNDER_BORROW_CAP_BPS note as a continuation of the trailing comment on MAX_EXERCISE_FEE_BPS and indented it to column 45, which is worse than what it replaced. It is now a plain block comment at the margin, which is what it always was in intent. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Owner
Author
|
Parking this as a draft rather than closing it: it is kept as research, not as pending work. Status as of 2026-08-31: this branch reformats The branch |
OxToF
marked this pull request as draft
August 31, 2026 12:46
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
cargo fmt --all --checkhas been red onmain, so Anchor Build & Lint fails on every PR — including branches that touch no Rust at all. Both PRs merged today (#7, #8) showed a red check they had no part in, and the job never got as far as the build it is named for.A lint that is always red stops being read, which is the actual cost here.
Contents
Formatting only.
cargo checkis clean.One change is not rustfmt's output: rustfmt read the
FOUNDER_BORROW_CAP_BPSnote as a continuation of the trailing comment onMAX_EXERCISE_FEE_BPSand indented it to column 45 — worse than what it replaced. It is now a plain block comment at the margin, which is what it was in intent.🤖 Generated with Claude Code