Skip to content

fix: use Node 24 for npm trusted publishing#17

Merged
pilat merged 1 commit intomainfrom
fix/publish-node24
Apr 28, 2026
Merged

fix: use Node 24 for npm trusted publishing#17
pilat merged 1 commit intomainfrom
fix/publish-node24

Conversation

@pilat
Copy link
Copy Markdown
Owner

@pilat pilat commented Apr 28, 2026

npm trusted publishing (OIDC) requires npm 11.5.1+. Node 22 ships with npm 10.x, so the old npm install -g npm@latest step was needed — but it broke on recent GitHub runners.

Node 24 ships with npm 11.x natively. Same fix others applied in npm/cli#8730.

Summary by CodeRabbit

  • Chores
    • Updated the build and publish pipeline to use a newer Node.js version for improved compatibility and stability.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 28, 2026

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a61b69bd-3d89-43a3-a04c-f0a36b6d1229

📥 Commits

Reviewing files that changed from the base of the PR and between 5358d7f and ee710c2.

📒 Files selected for processing (1)
  • .github/workflows/publish.yml

📝 Walkthrough

Walkthrough

Updated the Node.js version in the GitHub Actions publish workflow from version 22 to 24 within the setup-node@v6 action configuration. All subsequent workflow steps remain unchanged.

Changes

Cohort / File(s) Summary
Workflow Configuration
.github/workflows/publish.yml
Updated Node.js version from 22 to 24 in actions/setup-node@v6 configuration.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 Hop, hop, hooray! We've upgraded with glee,
Node twenty-four's the new bunny spree,
From twenty-two hops, to twenty-four bounds,
Our workflows now dance on fresher grounds! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: updating Node.js version from 22 to 24 for npm trusted publishing support.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/publish-node24

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@pilat pilat merged commit e2220a2 into main Apr 28, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant