Skip to content

#58 update actions yaml#60

Merged
ienaga merged 2 commits intomainfrom
develop
Feb 5, 2026
Merged

#58 update actions yaml#60
ienaga merged 2 commits intomainfrom
develop

Conversation

@ienaga
Copy link
Member

@ienaga ienaga commented Feb 5, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 5, 2026 00:48
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request updates the GitHub Actions workflow for publishing npm packages by removing the env block that provides authentication credentials for npm publishing.

Changes:

  • Removed the env block containing NODE_AUTH_TOKEN from the npm publish step in the publish workflow

💡 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
Copy link

Copilot AI Feb 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing the NODE_AUTH_TOKEN environment variable will cause the npm publish step to fail due to missing authentication credentials.

When using actions/setup-node with registry-url, the NODE_AUTH_TOKEN environment variable is required to authenticate with the npm registry for publishing packages. The setup-node action creates an .npmrc file that references NODE_AUTH_TOKEN, and without this environment variable being set, npm publish will not have the necessary credentials to authenticate and publish the package.

To publish to npm successfully, the env block with NODE_AUTH_TOKEN should be retained.

Suggested change
- run: npm publish --access public
- run: npm publish --access public
env:
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}

Copilot uses AI. Check for mistakes.
@ienaga ienaga merged commit 527766c into main Feb 5, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant