Skip to content

Run rustfmt over the lending Anchor v1 copy - #162

Merged
mikemaccana merged 1 commit into
mainfrom
claude/rustfmt-lending-anchor-v1
Sep 23, 2026
Merged

mikemaccana merged 1 commit into
mainfrom
claude/rustfmt-lending-anchor-v1

Conversation

@mikemaccana

@mikemaccana mikemaccana commented Sep 23, 2026 •

Copy link
Copy Markdown
Collaborator

The Rust Lint workflow runs cargo fmt --check on the root workspace only. The anchor-v1/ copies are deliberately kept outside that workspace, so the formatting of finance/lending/anchor-v1 drifted without CI noticing.

This PR is the output of cargo fmt run in finance/lending/anchor-v1, and nothing else. The changes are line wrapping only, mostly long calls in the test helpers split across lines. No code changes.

Verification

  • cargo fmt --check in finance/lending/anchor-v1 fails on main and is clean on this branch.
  • cargo test --no-run in that workspace compiles the program and every test target.

Not in this PR

The same drift exists elsewhere. Running cargo fmt --check in each of the other 55 anchor-v1/ workspaces on main fails in 35 of them, including the escrow, betting market, order book and token fundraiser copies under finance/. CI checks none of them. Formatting them and adding a cargo fmt --check step for the anchor-v1/ workspaces would each be a separate change.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Bb9YgXxuvSJThr3TjtDoca

The root Rust Lint workflow runs `cargo fmt --check` on the root workspace
only, and the `anchor-v1/` copies are deliberately outside it, so this copy's
formatting drifted unnoticed. This is `cargo fmt` run in
finance/lending/anchor-v1 and nothing else: line wrapping only, no code
changes. The workspace and its tests compile, and `cargo fmt --check` there
is now clean.

Claude-Session: https://claude.ai/code/session_01Bb9YgXxuvSJThr3TjtDoca
@mikemaccana
mikemaccana merged commit 2315d51 into main Sep 23, 2026
32 checks passed
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.

1 participant