Skip to content

chore: fix npm publish#141

Closed
fengmk2 wants to merge 1 commit intomainfrom
09-17-chore_fix_npm_publish
Closed

chore: fix npm publish#141
fengmk2 wants to merge 1 commit intomainfrom
09-17-chore_fix_npm_publish

Conversation

@fengmk2
Copy link
Copy Markdown
Member

@fengmk2 fengmk2 commented Sep 17, 2025

No description provided.

Copy link
Copy Markdown
Member Author

fengmk2 commented Sep 17, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@fengmk2 fengmk2 force-pushed the 09-17-chore_fix_npm_publish branch 4 times, most recently from 9e17291 to 3a6e7e2 Compare September 17, 2025 04:35
@fengmk2 fengmk2 force-pushed the 09-17-chore_fix_npm_publish branch from 3a6e7e2 to 2f2d201 Compare September 17, 2025 05:06
@fengmk2 fengmk2 marked this pull request as ready for review September 17, 2025 05:06
Copilot AI review requested due to automatic review settings September 17, 2025 05:06
Copy link
Copy Markdown
Contributor

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 PR fixes the npm publish workflow by updating the GitHub Actions authentication method and adding a temporary push trigger for testing.

  • Replaces manual npm authentication configuration with setup-node action authentication
  • Adds a temporary push trigger on a specific branch for testing the fix
  • Updates environment variable from NPM_TOKEN to NODE_AUTH_TOKEN

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines +6 to +7
branches:
- 09-17-chore_fix_npm_publish
Copy link

Copilot AI Sep 17, 2025

Choose a reason for hiding this comment

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

This branch-specific push trigger appears to be for testing purposes only. Consider removing this temporary trigger before merging to avoid unintended workflow executions on this specific branch in the future.

Suggested change
branches:
- 09-17-chore_fix_npm_publish
# branches: # Removed temporary branch-specific trigger
# - 09-17-chore_fix_npm_publish

Copilot uses AI. Check for mistakes.
@fengmk2 fengmk2 self-assigned this Sep 17, 2025
@Boshen Boshen closed this in 7e72fd4 Sep 17, 2025
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.

2 participants