Skip to content

ci(cd): pin installed npm version - #603

Merged
Fdawgs merged 3 commits into
mainfrom
ci/cd
Jul 12, 2026
Merged

ci(cd): pin installed npm version#603
Fdawgs merged 3 commits into
mainfrom
ci/cd

Conversation

@Fdawgs

@Fdawgs Fdawgs commented Jul 12, 2026

Copy link
Copy Markdown
Owner

'latest' is v12 which dropped support for node 20, which this package is still supporting for time being. Need v11 for OIDC publishing.

Copilot AI review requested due to automatic review settings July 12, 2026 09:41

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the CD workflow to avoid installing npm@latest (now v12, which drops Node 20 support) by pinning npm to a Node-20-compatible v11 release, ensuring the existing Node 20 publishing pipeline (including OIDC/provenance publishing) continues to work.

Changes:

  • Replace npm i -g npm@latest with npm i -g npm@11.18.0 --ignore-scripts in both publish jobs.
  • Keep the rest of the publish/build steps unchanged while ensuring npm stays compatible with the .nvmrc Node 20 runtime.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Fdawgs
Fdawgs merged commit a8eab7c into main Jul 12, 2026
20 checks passed
@Fdawgs
Fdawgs deleted the ci/cd branch July 12, 2026 10:28
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