fix(ci): pin npm bump to v10 to match node 20 engine requirement#592
fix(ci): pin npm bump to v10 to match node 20 engine requirement#592SteinGabriel wants to merge 1 commit into
Conversation
|
|
View your CI Pipeline Execution ↗ for commit b19d213
💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗ ☁️ Nx Cloud last updated this comment at |
commit: |
vatsalparikh
left a comment
There was a problem hiding this comment.
As far as I remember, npm 10 has issues during publishing or something. I will let Ryan review this. He has had to update npm before to 11.
Summary
CI publish workflow started failing on
developafter last merge..github/actions/setup-publish/action.ymlrunsnpm install -g npm@latestto update npm for provenancesigning, but npm just cut a new major (12.x) requiring
Node ^22.22.2 || ^24.15.0 || >=26.0.0. Workflowpins Node via
.node-version(currently20), so runner has Node 20.20.2 and install fails withEBADENGINE.Fix
Pin the npm bump to
npm@10(latest major still compatible with Node 20's engine range), instead offloating to
latest.Notes / Risks
10.xpatches.redundant entirely. Left as-is for this fix, worth reconsidering separately.