Skip to content

Commit e6f7fb7

Browse files
committed
ci(nodejs): remove pnpm version set and add publish dry run tests
Signed-off-by: peefy <[email protected]>
1 parent 263fcf3 commit e6f7fb7

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

.github/workflows/nodejs-test.yaml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,6 @@ jobs:
6666

6767
linux:
6868
runs-on: ubuntu-latest
69-
if: "startsWith(github.ref, 'refs/tags/')"
7069
strategy:
7170
matrix:
7271
settings:
@@ -129,7 +128,6 @@ jobs:
129128

130129
windows:
131130
runs-on: windows-latest
132-
if: "startsWith(github.ref, 'refs/tags/')"
133131
strategy:
134132
matrix:
135133
settings:
@@ -172,7 +170,6 @@ jobs:
172170

173171
macos:
174172
runs-on: macos-latest
175-
if: "startsWith(github.ref, 'refs/tags/')"
176173
strategy:
177174
matrix:
178175
settings:
@@ -262,6 +259,13 @@ jobs:
262259
run: ls -R ./npm
263260
shell: bash
264261

262+
- name: Publish Dry Run
263+
if: "startsWith(github.ref, 'refs/tags/') && contains(github.ref, '-')"
264+
# Since this command will not exit with non-zero code when file missing,
265+
# we need to check the output manually.
266+
run: |
267+
npm publish --access public --provenance --dry-run
268+
265269
- name: Publish
266270
if: "startsWith(github.ref, 'refs/tags/')"
267271
run: |

nodejs/package.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,5 @@
6868
"publishConfig": {
6969
"registry": "https://registry.npmjs.org/",
7070
"access": "public"
71-
},
72-
"packageManager": "[email protected]"
71+
}
7372
}

0 commit comments

Comments
 (0)