Skip to content

Commit c23faad

Browse files
committed
ci: Fix order in publish job
1 parent d2685a7 commit c23faad

File tree

1 file changed

+9
-10
lines changed

1 file changed

+9
-10
lines changed

.github/workflows/publish.yml

+9-10
Original file line numberDiff line numberDiff line change
@@ -33,16 +33,6 @@ jobs:
3333
node-version: 16.x
3434
registry-url: https://registry.npmjs.org
3535

36-
- name: Build
37-
run: npm run build
38-
39-
- name: Publish new version
40-
# Note: Setting NODE_AUTH_TOKEN as job|workspace wide env var won't work
41-
# as it appears actions/setup-node sets own value
42-
env:
43-
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
44-
run: npm publish
45-
4636
# Normally we have a guarantee that deps are already there, still it may not be the case when:
4737
# - `master` build for same commit failed (and we still pushed tag manually)
4838
# - We've pushed tag manually before `master` build finalized
@@ -52,6 +42,15 @@ jobs:
5242
npm update --no-save
5343
npm update --save-dev --no-save
5444
45+
- name: Build
46+
run: npm run build
47+
48+
- name: Publish new version
49+
# Note: Setting NODE_AUTH_TOKEN as job|workspace wide env var won't work
50+
# as it appears actions/setup-node sets own value
51+
env:
52+
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
53+
run: npm publish
5554
- name: Publish release notes
5655
run: |
5756
TEMP_ARRAY=($(echo $GITHUB_REF | tr "/" "\n"))

0 commit comments

Comments
 (0)