Skip to content

ci: add setup-node@v4 with Node 22 before npm ci in release workflow#81

Merged
zhukaihan merged 2 commits intocodex/release-lockfilesfrom
copilot/sub-pr-80
Mar 20, 2026
Merged

ci: add setup-node@v4 with Node 22 before npm ci in release workflow#81
zhukaihan merged 2 commits intocodex/release-lockfilesfrom
copilot/sub-pr-80

Conversation

Copy link

Copilot AI commented Mar 20, 2026

The release workflow was running npm ci without a pinned Node/npm version, risking failures when the runner image changes — particularly since package-lock.json uses lockfileVersion: 3 (requires npm v9+).

Changes

  • Added actions/setup-node@v4 step with node-version: 22 and cache: 'npm' in .github/workflows/release.yml, placed before both npm ci invocations

Summary

Pins Node 22 in the release workflow to ensure a compatible npm version is always available for the committed lockfile.

Checklist

  • Does your PR title have the correct title format?
  • Does your PR have a breaking change?: No

📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

Copilot AI changed the title [WIP] [WIP] Address feedback: Use Node 22 for NPM dependency setup ci: add setup-node@v4 with Node 22 before npm ci in release workflow Mar 20, 2026
Copilot AI requested a review from zhukaihan March 20, 2026 21:49
@zhukaihan zhukaihan marked this pull request as ready for review March 20, 2026 21:50
@zhukaihan zhukaihan merged commit 6a58ec9 into codex/release-lockfiles Mar 20, 2026
1 check passed
@zhukaihan zhukaihan deleted the copilot/sub-pr-80 branch March 20, 2026 21:50
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.

2 participants