Skip to content

(ci) Stop explicitly using the latest npm - no longer needed#17408

Merged
turbobobbytraykov merged 1 commit into
masterfrom
btraykov/master-no-explicit-new-npm
Jul 10, 2026
Merged

(ci) Stop explicitly using the latest npm - no longer needed#17408
turbobobbytraykov merged 1 commit into
masterfrom
btraykov/master-no-explicit-new-npm

Conversation

@turbobobbytraykov

@turbobobbytraykov turbobobbytraykov commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Removed the step to update NPM to the latest version. Node 24 ships with a good enough version of npm in order to make Trusted Publishing to npmjs.org work.

Closes #
N/A

Description

Having this step was mandatory for node 20 and earlier builds of 22, but 24 ships a good enough node cli client.

Motivation / Context

The latest compatible version of the npm cli client ships with a show-stopper bug right now: npm/cli#9722 and we want to avoid hitting it.

Type of Change (check all that apply):

  • Bug fix
  • New functionality
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactoring (no functional changes)
  • Documentation
  • Demos
  • [x ] CI/CD
  • Tests
  • Changelog
  • Skills/Agents

Component(s) / Area(s) Affected:

N/A

How Has This Been Tested?

N/A

Test Configuration:

N/A

Screenshots / Recordings

N/A

Checklist:

  • All relevant tags have been applied to this PR
  • This PR includes unit tests covering all the new code (test guidelines)
  • This PR includes API docs for newly added methods/properties (api docs guidelines)
  • This PR includes feature/README.MD updates for the feature docs
  • This PR includes general feature table updates in the root README.MD
  • This PR includes CHANGELOG.MD updates for newly added functionality
  • This PR contains breaking changes
  • This PR includes ng update migrations for the breaking changes (migrations guidelines)
  • This PR includes behavioral changes and the feature specification has been updated with them
  • Accessibility (ARIA, keyboard navigation, focus management) has been verified

Removed the step to update NPM to the latest version.
Copilot AI review requested due to automatic review settings July 10, 2026 08:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 simplifies the npm publish GitHub Actions workflow by removing the step that force-upgraded npm to latest, relying instead on the npm version bundled with Node.js 24 (to avoid a known npm CLI regression and keep Trusted Publishing working).

Changes:

  • Removed the npm install -g npm@latest step from the release publish workflow.

@turbobobbytraykov turbobobbytraykov merged commit c4530b5 into master Jul 10, 2026
7 checks passed
@turbobobbytraykov turbobobbytraykov deleted the btraykov/master-no-explicit-new-npm branch July 10, 2026 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants