Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR simplifies the npm publishing workflow by removing redundant environment variable configuration. The NODE_AUTH_TOKEN environment variable is no longer explicitly set for the publish step, likely because it's already configured globally in the workflow or handled by the setup-node action.
Key Changes
- Removed explicit
NODE_AUTH_TOKENenvironment variable declaration from the publish step
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - run: npm publish --access public | ||
| env: | ||
| NODE_AUTH_TOKEN: ${{ secrets.NODE_AUTH_TOKEN }} No newline at end of file | ||
| - run: npm publish --access public No newline at end of file |
There was a problem hiding this comment.
The NODE_AUTH_TOKEN environment variable has been removed, but npm publish requires authentication. Without this token, the publish step will fail unless the token is configured elsewhere in the workflow. Verify that setup-node with registry-url automatically provides authentication, or restore the env block with NODE_AUTH_TOKEN: ${{ secrets.NODE_AUTH_TOKEN }}.
| - run: npm publish --access public | |
| - run: npm publish --access public | |
| env: | |
| NODE_AUTH_TOKEN: ${{ secrets.NODE_AUTH_TOKEN }} |
No description provided.