Skip to content

fix(bb-sol): commit Debug.sol in forge-fmt-stable form on next#23168

Merged
alexghr merged 1 commit into
nextfrom
claudebox/656b4ce2b1e5e7de-2
May 12, 2026
Merged

fix(bb-sol): commit Debug.sol in forge-fmt-stable form on next#23168
alexghr merged 1 commit into
nextfrom
claudebox/656b4ce2b1e5e7de-2

Conversation

@AztecBot
Copy link
Copy Markdown
Collaborator

@AztecBot AztecBot commented May 12, 2026

Summary

Companion to #23146 (merge-train/barretenberg), recreated against next.

barretenberg/sol/bootstrap.sh::build_sol runs forge fmt before forge build. PR #22659 (now in next via merge-train/fairies) trimmed unused imports from test/utils/Debug.sol; once trimmed, the multi-line import block was short enough that forge fmt collapses it to a single line. That rewrites the file mid-CI and trips ci3/cache_content_hash with ERROR: Noticed changes to rebuild patterns during CI run, disabling cache hits for every downstream *-tests target — as seen in http://ci.aztec-labs.com/1778494208886637.

This PR puts Debug.sol directly into the forge-fmt-stable form (single-line import). After rebase, this is the only diff against next.

Verification

cd barretenberg/sol
forge fmt --check    # clean
forge fmt            # no changes

Full analysis: https://gist.github.com/AztecBot/86bb7e59c1a9998f77dd1bc51dd9319d

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels May 12, 2026
@AztecBot AztecBot force-pushed the claudebox/656b4ce2b1e5e7de-2 branch from 905d4ae to 18f3fa8 Compare May 12, 2026 08:13
@alexghr alexghr marked this pull request as ready for review May 12, 2026 08:17
@alexghr alexghr enabled auto-merge May 12, 2026 08:17
@alexghr alexghr added this pull request to the merge queue May 12, 2026
Merged via the queue into next with commit 32b9d85 May 12, 2026
27 of 33 checks passed
@alexghr alexghr deleted the claudebox/656b4ce2b1e5e7de-2 branch May 12, 2026 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants