Skip to content

Commit efaf189

Browse files
authored
Merge pull request #7 from charfeng1/fix/ci-publish-node-24
fix(ci): use Node 24 for publish (skip broken npm self-upgrade)
2 parents 825845e + e6cb007 commit efaf189

1 file changed

Lines changed: 5 additions & 6 deletions

File tree

.github/workflows/release.yml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -68,19 +68,18 @@ jobs:
6868
# previous tag to compute changes against.
6969
fetch-depth: 0
7070

71+
# Node 24 ships npm 11.x, which satisfies trusted publishing's
72+
# >=11.5.1 requirement without a self-upgrade step (the self-upgrade
73+
# has a known `Cannot find module 'promise-retry'` bug on top of
74+
# setup-node's pre-installed npm).
7175
- uses: actions/setup-node@v4
7276
with:
73-
node-version: 22
77+
node-version: 24
7478
registry-url: 'https://registry.npmjs.org'
7579
cache: npm
7680

7781
- run: npm ci
7882

79-
# Trusted publishing requires npm >= 11.5.1. setup-node@v4 ships an
80-
# older npm by default, so upgrade before publish.
81-
- name: Upgrade npm
82-
run: npm install -g npm@latest
83-
8483
- name: Detect version change
8584
id: check
8685
run: |

0 commit comments

Comments
 (0)