Skip to content

Commit b15373d

Browse files
bugerclaude
andcommitted
ci: update release workflow to match probe's npm publishing setup
- Use actions/checkout@v4 and actions/setup-node@v4 (not pinned SHAs) - Update to Node.js 24 - Remove rollup workaround (fixed in newer npm) - Simplify install steps Co-Authored-By: Claude Opus 4.5 <[email protected]>
1 parent 1b62452 commit b15373d

File tree

1 file changed

+13
-33
lines changed

1 file changed

+13
-33
lines changed

.github/workflows/release.yml

Lines changed: 13 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -28,23 +28,16 @@ jobs:
2828

2929
steps:
3030
- name: Checkout code
31-
uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
31+
uses: actions/checkout@v4
3232

3333
- name: Setup Node.js
34-
uses: actions/setup-node@1e60f620b9541d16bece96c5465dc8ee9832be0b # v4.0.3
34+
uses: actions/setup-node@v4
3535
with:
36-
node-version: '20'
37-
registry-url: 'https://registry.npmjs.org'
38-
# cache: 'npm' # Disabled due to rollup optional dependency bug
36+
node-version: "24"
37+
registry-url: "https://registry.npmjs.org"
3938

4039
- name: Install dependencies
41-
run: |
42-
# Workaround for npm bug with rollup optional dependencies
43-
# https://github.com/npm/cli/issues/4828
44-
rm -rf node_modules package-lock.json
45-
npm install
46-
# Explicitly install rollup platform-specific binary
47-
npm install --no-save @rollup/rollup-linux-x64-gnu
40+
run: npm install
4841

4942
- name: Run tests
5043
run: npm test
@@ -104,24 +97,18 @@ jobs:
10497

10598
steps:
10699
- name: Checkout code
107-
uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
100+
uses: actions/checkout@v4
108101
with:
109102
fetch-depth: 0 # Need full history for git log
110103

111104
- name: Setup Node.js
112-
uses: actions/setup-node@1e60f620b9541d16bece96c5465dc8ee9832be0b # v4.0.3
105+
uses: actions/setup-node@v4
113106
with:
114-
node-version: '20'
115-
# cache: 'npm' # Disabled due to rollup optional dependency bug
107+
node-version: "24"
116108

117109
- name: Install dependencies and build
118110
run: |
119-
# Workaround for npm bug with rollup optional dependencies
120-
# https://github.com/npm/cli/issues/4828
121-
rm -rf node_modules package-lock.json
122111
npm install
123-
# Explicitly install rollup platform-specific binary
124-
npm install --no-save @rollup/rollup-linux-x64-gnu
125112
npm run build
126113
127114
- name: Set version variables
@@ -249,23 +236,16 @@ jobs:
249236

250237
steps:
251238
- name: Checkout code
252-
uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
239+
uses: actions/checkout@v4
253240

254241
- name: Setup Node.js
255-
uses: actions/setup-node@1e60f620b9541d16bece96c5465dc8ee9832be0b # v4.0.3
242+
uses: actions/setup-node@v4
256243
with:
257-
node-version: '20'
258-
registry-url: 'https://registry.npmjs.org'
259-
# cache: 'npm' # Disabled due to rollup optional dependency bug
244+
node-version: "24"
245+
registry-url: "https://registry.npmjs.org"
260246

261247
- name: Install dependencies
262-
run: |
263-
# Workaround for npm bug with rollup optional dependencies
264-
# https://github.com/npm/cli/issues/4828
265-
rm -rf node_modules package-lock.json
266-
npm install
267-
# Explicitly install rollup platform-specific binary
268-
npm install --no-save @rollup/rollup-linux-x64-gnu
248+
run: npm install
269249

270250
- name: Build package
271251
run: npm run build

0 commit comments

Comments
 (0)