Skip to content

Conversation

@erezrokah
Copy link
Member

Removed NODE_AUTH_TOKEN environment variable from npm publish step.

I've set up trusted publishing so no need for the token.

See https://docs.npmjs.com/trusted-publishers#step-1-add-a-trusted-publisher-on-npmjscom.

Ended up doing this since got a notification our token expired

Add permission for id-token and remove NODE_AUTH_TOKEN from npm publish step.
@erezrokah erezrokah requested a review from a team as a code owner February 2, 2026 11:44
@erezrokah erezrokah requested review from stoovon and removed request for a team February 2, 2026 11:44
@erezrokah erezrokah added the automerge Add to automerge PRs once requirements are met label Feb 2, 2026
@kodiakhq kodiakhq bot merged commit 6ae51d1 into main Feb 2, 2026
9 checks passed
@kodiakhq kodiakhq bot deleted the erezrokah-patch-1 branch February 2, 2026 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Add to automerge PRs once requirements are met

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants