Skip to content

Commit

Permalink
try to fix pnpm publish
Browse files Browse the repository at this point in the history
  • Loading branch information
webzwo0i committed Apr 14, 2024
1 parent 9f2d5a5 commit d6c8100
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions .github/workflows/npmpublish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,12 @@ jobs:
# in dependencies or devDependencies.
-
run: pnpm i
# Workaround based on https://github.com/pnpm/pnpm/issues/3141
- name: Set publishing config
run: pnpm config set '//registry.npmjs.org/:_authToken' "${NODE_AUTH_TOKEN}"
env:
NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}}

# `npm publish` must come after `git push` otherwise there is a race
# condition: If two PRs are merged back-to-back then master/main will be
# updated with the commits from the second PR before the first PR's
Expand All @@ -65,8 +71,6 @@ jobs:
# the second's will succeed.
-
run: pnpm publish
env:
NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}}
#-
# name: Add package to etherpad organization
# run: pnpm access grant read-write etherpad:developers
Expand Down

0 comments on commit d6c8100

Please sign in to comment.