fix: pin noir submodule to next's version on merge-train/spartan#23690
Merged
Conversation
PhilWindle
approved these changes
May 29, 2026
danielntmd
pushed a commit
to danielntmd/aztec-packages
that referenced
this pull request
Jun 4, 2026
BEGIN_COMMIT_OVERRIDE test(e2e): unskip pipelining related e2e tests (AztecProtocol#23642) fix(archiver): prune blocks without proposed checkpoint by end of build slot (AztecProtocol#23606) test: migrate benchmarks to pipelining setup (AztecProtocol#23647) fix(p2p): fall back to archiver in BLOCK_TXS response validation (AztecProtocol#23624) docs(slashing): align operator and slasher docs with AZIP-7 (AztecProtocol#23494) fix(p2p): do not penalize peers that signal a missing block with Fr.ZERO (AztecProtocol#23672) chore: adjust metrics deployment (AztecProtocol#23676) fix(cheat-codes): warpL2TimeAtLeastBy advances relative to leading clock (AztecProtocol#23675) chore: tighten node pool sizes (AztecProtocol#23678) chore: remove archival nodes (AztecProtocol#23630) chore: merge blob sink duties into RPC node (AztecProtocol#23631) fix: sync avm-transpiler Cargo.lock with noir submodule (AztecProtocol#23683) fix(spartan): set validator lag env vars in tps-scenario (AztecProtocol#23684) fix: make world-state hash queries reorg-aware to close getWorldState race (AztecProtocol#23677) fix: pin noir submodule to next's version on merge-train/spartan (AztecProtocol#23690) fix: ensure image ref is used by bench runner (AztecProtocol#23682) fix(ci): retry aztec-nr nargo dependency clone on transient network flake (AztecProtocol#23653) chore: run one-off jobs on network nodes (AztecProtocol#23701) fix: simulate proposals inside target slot (AztecProtocol#23692) chore: smaller eth-devnet (AztecProtocol#23704) chore: enable testnet autoscaling (AztecProtocol#23705) feat(api)!: redesign node log retrieval API around tag-based queries (AztecProtocol#23625) fix(sequencer): set own proposed checkpoint locally instead of via p2p loopback (AztecProtocol#23659) END_COMMIT_OVERRIDE
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
2 participants
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.
Problem
CI on
merge-train/spartanis failing in theaztec-nrstep (log) withBoundedVec::from_parts_uncheckeddeprecation errors undernargo check --deny-warnings.The train's noir submodule had diverged from
next:from_parts_uncheckeddeprecated?nextf1a4575merge-train/spartan4d039268The newer pin (
4d039268) was pulled onto the train by PR #23675 ("fix(cheat-codes): warpL2TimeAtLeastBy…"), which bumpednoir/noir-repofrom the May-11 pin to a May-28 nightly. That nightly added#[deprecated]toBoundedVec::from_parts_unchecked, and since aztec-nr builds with--deny-warnings, the two remaining call sites became hard errors. The two noir commits are on divergent lines (neither is an ancestor of the other), so the train was simply ahead ofnexton noir.Fix
Pin the train's noir back to exactly what
nextuses. Only two files differed fromnext:noir/noir-repo→f1a4575(next's pin)avm-transpiler/Cargo.lock→ next's version (it had been re-synced to the May-28 noir by fix: sync avm-transpiler Cargo.lock with noir submodule #23683; restored to match the May-11 pin)This restores parity with
nextand removes the deprecated API entirely, so no aztec-nr source change is needed.Verification
Built
nargofrom thef1a4575pin and ran the failing check against the unmodified aztec-nr source:nargo check --deny-warnings→ exit 0 (the deprecation attribute is absent inf1a4575).Note
This is an alternative to #23687, which fixed the same failure by patching the two aztec-nr call sites to use
from_partsagainst the newer noir. Pick one: this PR keeps the train aligned withnext's noir; #23687 keeps the newer noir and updates the source. Closing whichever isn't chosen.Created by claudebox · group:
slackbot