Skip to content

chore(ci): remove redundant npm@latest upgrade#126

Merged
mscolnick merged 1 commit into
mainfrom
myles/remove-npm-latest
May 12, 2026
Merged

chore(ci): remove redundant npm@latest upgrade#126
mscolnick merged 1 commit into
mainfrom
myles/remove-npm-latest

Conversation

@mscolnick
Copy link
Copy Markdown
Contributor

node-24 ships with npm 11+ which already supports OIDC trusted publishing, so the npm install -g npm@latest step is redundant. Removing it also drops one global install in the same job as id-token: write.

Copilot AI review requested due to automatic review settings May 12, 2026 16:52
@mscolnick mscolnick merged commit e8fe76d into main May 12, 2026
8 checks passed
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes a redundant npm upgrade step from the release GitHub Actions workflow, relying on the npm version bundled with Node.js 24 for OIDC/trusted publishing support.

Changes:

  • Removed the “Upgrade npm for OIDC support” step (pnpm install -g npm@latest) from the release workflow.

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

@mscolnick mscolnick deleted the myles/remove-npm-latest branch May 12, 2026 16:54
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