Skip to content

Commit 0cdf5c5

Browse files
committed
[fix] release
1 parent 353785f commit 0cdf5c5

1 file changed

Lines changed: 6 additions & 7 deletions

File tree

‎.github/workflows/release.yml‎

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,15 @@ jobs:
2525
steps:
2626
- uses: actions/checkout@v4
2727

28+
# Node 24 ships npm 11.x, which already supports Trusted Publishers
29+
# (OIDC). We intentionally don't run `npm install -g npm@latest` — the
30+
# npm bundled with some Node 22.x patch releases has a broken
31+
# node_modules (missing `promise-retry`) that makes self-upgrade fail.
32+
# ci.yml still runs the test matrix against 18/20/22 for runtime compat.
2833
- name: Setup Node.js
2934
uses: actions/setup-node@v4
3035
with:
31-
node-version: 22.x
36+
node-version: 24.x
3237
registry-url: https://registry.npmjs.org
3338
cache: npm
3439

@@ -42,12 +47,6 @@ jobs:
4247
NOMINAL_OAUTH_CLIENT_ID: ${{ secrets.NOMINAL_OAUTH_CLIENT_ID }}
4348
NOMINAL_OAUTH_CLIENT_SECRET: ${{ secrets.NOMINAL_OAUTH_CLIENT_SECRET }}
4449

45-
# npm Trusted Publishers (OIDC) needs npm >= 11.5.1; node 22.x ships with
46-
# an older npm. Upgrade before publishing so the CLI picks up the OIDC
47-
# auth flow and no NPM_TOKEN is required.
48-
- name: Upgrade npm for trusted publishing
49-
run: npm install -g npm@latest
50-
5150
- name: Publish to npm
5251
run: npm publish --access public --provenance
5352

0 commit comments

Comments
 (0)