Skip to content

fix: noir format in compute_note_hash_and_nullifier.nr#21956

Merged
Thunkar merged 1 commit into
backport-to-v4-next-stagingfrom
claudebox/fix-noir-format-21899
Mar 24, 2026
Merged

fix: noir format in compute_note_hash_and_nullifier.nr#21956
Thunkar merged 1 commit into
backport-to-v4-next-stagingfrom
claudebox/fix-noir-format-21899

Conversation

@AztecBot

Copy link
Copy Markdown
Collaborator

Summary

Fixes nargo fmt --check failure on the backport branch by wrapping two long comment lines in compute_note_hash_and_nullifier.nr that exceeded the formatter's line length limit.

The backport of the resolve_fn changes fixed a typo in two comments ("call we correct" → "call the correct"), which pushed those lines past the formatter threshold. This wraps them to satisfy nargo fmt.

ClaudeBox log: https://claudebox.work/s/338e316c36d5c83e?run=1

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels Mar 24, 2026
@Thunkar Thunkar marked this pull request as ready for review March 24, 2026 14:56
@Thunkar Thunkar requested a review from nventuro as a code owner March 24, 2026 14:56
@Thunkar Thunkar self-requested a review March 24, 2026 14:57
@Thunkar Thunkar enabled auto-merge (squash) March 24, 2026 14:58
@AztecBot

Copy link
Copy Markdown
Collaborator Author

Flakey Tests

🤖 says: This CI run detected 2 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/9386d9db40a16edc�9386d9db40a16edc8;;�):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_epochs/epochs_mbps.parallel.test.ts "builds multiple blocks per slot with transactions anchored to checkpointed block" (177s) (code: 0) group:e2e-p2p-epoch-flakes
\033FLAKED\033 (8;;http://ci.aztec-labs.com/9afbb7747c765cb1�9afbb7747c765cb18;;�):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_epochs/epochs_proof_fails.parallel.test.ts "does not allow submitting proof after epoch end" (219s) (code: 0) group:e2e-p2p-epoch-flakes

@Thunkar Thunkar merged commit c85d12c into backport-to-v4-next-staging Mar 24, 2026
30 of 36 checks passed
@Thunkar Thunkar deleted the claudebox/fix-noir-format-21899 branch March 24, 2026 15:16
AztecBot added a commit that referenced this pull request Mar 25, 2026
BEGIN_COMMIT_OVERRIDE
fix(aztec-up): narrow PATH cleanup regex to avoid removing user PATH
entries (#21828)
fix: consolidate blob source test into single summary log with supernode
detection (#21719)
fix: use anchor block on getL1ToL2MsgWitness (#21872)
fix: make sure queries are not made ahead of the anchor block (#21874)
chore(docs): backport docs infrastructure changes to v4 (#21437)
chore: backport PR #21836 (fix: explicitly handle initial block case for
getBlockHashMembershipWitness) to v4-next (#21891)
chore: backport PR #21813 (fix(aztec-up): strip leading v prefix from
version strings) to v4-next (#21912)
docs: document defi-wonderland immutable macro (#21764)
fix: pin typescript and harden lockfile check in docs examples CI (v4
backport) (#21914)
docs: expand circuit profiling guide and document kernel gate cost
overhead (#21770)
feat: cli-wallet (#21757)
chore: sync backport staging with v4-next HEAD (#21931)
fix: bot gas estimations (backport #21945) (#21947)
refactor(aztec-nr): use logging wrappers in
compute_note_hash_and_nullifier (#21897)
fix: backport v4 next staging (#21934)
fix: make PXE#getSyncedBlockHeader a concurrency=1 job to prevent IDB tx
liveness issues (#21944)
feat: add error page mapping for incompatible oracles (#21943)
fix(aztec-up): validate semver in uninstall to prevent path traversal
(#21892)
chore: merge v4-next into backport staging (fix ancestry) (#21954)
fix(aztec-up): add truncation protection to install scripts (#21896)
fix: noir format in compute_note_hash_and_nullifier.nr (#21956)
fix(stdlib): decode `EthAddress`, `FunctionSelector` and wrapped field
structs in `AbiDecoder` (#21926)
chore: update noir to nightly-2026-02-12-patch.1 (#21960)
fix: ensure empty docs examples does not populate yarn lock backport
(#21963)
fix: docs examples does not build on simple bootstrap backport (#21964)
fix: add rc/testnet version detection to API reference pages (#21966)
docs: document EmbeddedWallet automatic gas estimation and private
authwits (#21790)
fix(pxe): robust error handling in message processing (backport #21093)
(#21981)
chore: manually recreate backport (#21969)
chore: (A-779) load all accounts before calling
LogService.#getSecretsForSenders (#21923)
END_COMMIT_OVERRIDE
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.

2 participants