Skip to content

chore(clippy): fix workspace-wide clippy regressions#14554

Merged
grandizzy merged 8 commits intomasterfrom
steven/clippy-fixes
May 4, 2026
Merged

chore(clippy): fix workspace-wide clippy regressions#14554
grandizzy merged 8 commits intomasterfrom
steven/clippy-fixes

Conversation

@stevencartavia
Copy link
Copy Markdown
Collaborator

@stevencartavia stevencartavia commented May 4, 2026

Trivial clippy fixes for lints fired under recent nightly clippy across the workspace.

Lints addressed:

  • clippy::for_kv_mapcrates/common/src/contracts.rs
  • clippy::useless_borrows_in_formatting — drop redundant & on format!/write!/sh_println! arguments across:
    • crates/anvil/src/eth/pool/transactions.rs
    • crates/cast/src/cmd/wallet/mod.rs, crates/cast/src/lib.rs
    • crates/cheatcodes/src/test/assert.rs
    • crates/chisel/src/executor.rs
    • crates/doc/src/writer/as_doc.rs
    • crates/evm/core/src/decode.rs
    • crates/evm/evm/src/executors/invariant/mod.rs
    • crates/evm/fuzz/src/lib.rs
    • crates/fmt/src/state/mod.rs
    • crates/forge/src/cmd/test/mod.rs, crates/forge/src/cmd/test/summary.rs, crates/forge/src/gas_report.rs, crates/forge/src/runner.rs
    • crates/script/src/verify.rs
  • clippy::uninlined_format_args — inline format args at two cast wallet messages exposed after removing the borrows.

No behavior change.

grandizzy
grandizzy previously approved these changes May 4, 2026
@grandizzy grandizzy enabled auto-merge (squash) May 4, 2026 05:03
@stevencartavia stevencartavia changed the title chore(clippy): fix for_kv_map and useless_borrows_in_formatting chore(clippy): fix workspace-wide clippy regressions May 4, 2026
@grandizzy grandizzy self-requested a review May 4, 2026 05:58
@grandizzy grandizzy merged commit 84d3873 into master May 4, 2026
18 checks passed
@grandizzy grandizzy deleted the steven/clippy-fixes branch May 4, 2026 06:00
@github-project-automation github-project-automation Bot moved this to Done in Foundry May 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants