Skip to content

fix: install node 22 for aztec-cli acceptance test#22917

Merged
nchamo merged 1 commit into
merge-train/fairiesfrom
aztec-cli-acceptance-fix
May 4, 2026
Merged

fix: install node 22 for aztec-cli acceptance test#22917
nchamo merged 1 commit into
merge-train/fairiesfrom
aztec-cli-acceptance-fix

Conversation

@nchamo
Copy link
Copy Markdown
Contributor

@nchamo nchamo commented May 3, 2026

Summary

  • The acceptance test runs a .ts harness via run-test.sh that relies on Node's TS type-stripping, which requires Node >=22.18.
  • The runner's default Node was too old, so the workflow now installs Node 22 before running the test. The aztec CLI installer continues to manage its own Node version independently.

@nchamo nchamo requested a review from charlielye as a code owner May 3, 2026 23:09
@nchamo nchamo self-assigned this May 3, 2026
@nchamo nchamo changed the title ci: install node 22 for aztec-cli acceptance test fix: install node 22 for aztec-cli acceptance test May 3, 2026
@nchamo nchamo requested a review from mverzilli May 3, 2026 23:15
@nchamo nchamo enabled auto-merge (squash) May 4, 2026 12:17
@nchamo nchamo merged commit da530f9 into merge-train/fairies May 4, 2026
26 of 27 checks passed
@nchamo nchamo deleted the aztec-cli-acceptance-fix branch May 4, 2026 12:17
@AztecBot
Copy link
Copy Markdown
Collaborator

AztecBot commented May 4, 2026

✅ Successfully backported to backport-to-v4-next-staging #22924.

chrismarino pushed a commit to chrismarino/aztec-packages that referenced this pull request May 5, 2026
BEGIN_COMMIT_OVERRIDE
chore: fix kv-store browser tests hangs (AztecProtocol#22721)
feat: kv-store sqlite backend with page level encryption (AztecProtocol#22759)
fix: install node 22 for aztec-cli acceptance test (AztecProtocol#22917)
fix(kv-store): exclude @aztec/sqlite3mc-wasm from vitest optimizeDeps
(AztecProtocol#22929)
feat(txe): add tx private logs to tx side effects oracle (AztecProtocol#22889)
feat(aztec-nr): add call_self stubs for utility functions (AztecProtocol#22885)
END_COMMIT_OVERRIDE
AztecBot added a commit that referenced this pull request May 5, 2026
BEGIN_COMMIT_OVERRIDE
docs: add map and state variable docs  (#22824)
fix: e2e compat should not fail for contracts added after legacy stables
(#22900)
chore: fix kv-store browser tests hangs (#22721)
feat: kv-store sqlite backend with page level encryption (#22759)
fix: install node 22 for aztec-cli acceptance test (#22917)
feat: backport kv-store sqlite encryption (#22759) to v4-next (#22927)
fix(docs): correct llms.txt links for versioned developer docs (#22819)
feat(docs): improve discoverability of Aztec.nr API reference docs
(#22861)
feat(docs): backport improve discoverability of Aztec.nr API reference
docs (#22861) to v4-next (#22931)
feat(aztec-nr): add call_self stubs for utility functions (#22885)
docs: add map and state variable docs (backport #22824) (#22880)
refactor: `getPackageVersion` fn cleanup (#22941)
fix(ci): skip acceptance test for canary -commit. tags (#22951)
fix: closing db, correct stub side effects (#22939)
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants