Skip to content

ci: upgrade nodejs release to npm 12#7900

Merged
Xuanwo merged 2 commits into
mainfrom
xuanwo/fix-nodejs-release-npm-major
Jul 10, 2026
Merged

ci: upgrade nodejs release to npm 12#7900
Xuanwo merged 2 commits into
mainfrom
xuanwo/fix-nodejs-release-npm-major

Conversation

@Xuanwo

@Xuanwo Xuanwo commented Jul 10, 2026

Copy link
Copy Markdown
Member

Which issue does this PR close?

Closes #7897.

Rationale for this change

The NodeJS release workflow was running on Node 20 while installing npm@latest. During the 0.58.0 release, npm@latest resolved to npm 12, which requires a newer Node runtime and caused the publish job to fail.

Instead of pinning the release workflow to an older npm major, this PR upgrades the NodeJS release workflow to run on Node 24 and installs npm 12 explicitly.

What changes are included in this PR?

  • Upgrade the NodeJS release build and publish jobs from Node 20 to Node 24.
  • Install npm@12 in the publish job.
  • Update the release job name from node@20 to node@24.

Are there any user-facing changes?

No.

AI Usage Statement

AI assistance was used to prepare this PR.

@Xuanwo Xuanwo changed the title ci: pin npm major for nodejs release ci: upgrade nodejs release to npm 12 Jul 10, 2026
@Xuanwo Xuanwo marked this pull request as ready for review July 10, 2026 12:27
@Xuanwo Xuanwo requested a review from tisonkun as a code owner July 10, 2026 12:27
@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" labels Jul 10, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Jul 10, 2026
@Xuanwo Xuanwo deployed to release July 10, 2026 12:37 — with GitHub Actions Active
@Xuanwo Xuanwo merged commit aedc810 into main Jul 10, 2026
30 checks passed
@Xuanwo Xuanwo deleted the xuanwo/fix-nodejs-release-npm-major branch July 10, 2026 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

release: NodeJS publish should not install npm latest on Node 20

2 participants