Skip to content

Commit a76cc61

Browse files
authored
Canary publish fix 2 (#1338)
# why # what changed # test plan <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Simplified the canary publish step by removing manual pnpm auth setup in the release workflow. The job now relies on NODE_AUTH_TOKEN from the environment. ## Why: - The pnpm auth config was redundant and could add noisy logs. - Publishing already uses NODE_AUTH_TOKEN via env. ## What: - Removed pnpm config set //registry.npmjs.org/:_authToken=${NODE_AUTH_TOKEN} from the Publish Canary step. ## Test Plan: - [ ] Trigger the release workflow on main. - [ ] Verify canary publish completes successfully. - [ ] Confirm auth is picked up from NODE_AUTH_TOKEN. - [ ] Ensure no token value appears in logs. <sup>Written for commit d0908db. Summary will update automatically on new commits.</sup> <!-- End of auto-generated description by cubic. -->
1 parent cfa9c1a commit a76cc61

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

.github/workflows/release.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,6 @@ jobs:
4646
- name: Publish Canary
4747
if: github.ref == 'refs/heads/main'
4848
run: |
49-
pnpm config set //registry.npmjs.org/:_authToken=${NODE_AUTH_TOKEN}
5049
git checkout main
5150
pnpm run release-canary
5251
env:

0 commit comments

Comments
 (0)