Skip to content

Commit cf2ec69

Browse files
wemeetagainclaude
andauthored
fix: correct release workflow publish issues (#513)
## Summary - **Fix typo:** `release_created` (singular) → `releases_created` (plural) on the `pnpm/action-setup` step — this caused the step to silently be skipped on every release - **Add missing conditional** to `corepack enable` so it only runs when a release is created - **Add `registry-url`** to `actions/setup-node` so the OIDC token exchange can authenticate against npm for trusted publishing ## Test plan - [ ] Verify release workflow runs correctly on next release-please PR merge - [ ] Confirm npm packages are published with provenance 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 1d0ff62 commit cf2ec69

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

.github/workflows/release.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,14 +30,16 @@ jobs:
3030
if: ${{ steps.release.outputs.releases_created }}
3131

3232
- uses: pnpm/action-setup@v4
33-
if: ${{ steps.release.outputs.release_created }}
33+
if: ${{ steps.release.outputs.releases_created }}
3434

3535
- run: corepack enable
36-
36+
if: ${{ steps.release.outputs.releases_created }}
37+
3738
- uses: actions/setup-node@v6
3839
with:
3940
cache: 'pnpm'
4041
node-version: 20
42+
registry-url: 'https://registry.npmjs.org'
4143
if: ${{ steps.release.outputs.releases_created }}
4244

4345
- run: pnpm install --frozen-lockfile

0 commit comments

Comments
 (0)