Skip to content

Simplify npm publish step in workflow#47

Merged
TylerJDev merged 1 commit intomainfrom
gp/migrate-oidc
May 6, 2026
Merged

Simplify npm publish step in workflow#47
TylerJDev merged 1 commit intomainfrom
gp/migrate-oidc

Conversation

@gracepark
Copy link
Copy Markdown
Contributor

Removed npm whoami command from publish workflow as we migrated to OIDC. The NODE_AUTH_TOKEN is no longer needed.

Removed npm whoami command from publish workflow.
Copilot AI review requested due to automatic review settings May 5, 2026 19:06
@gracepark gracepark requested a review from a team as a code owner May 5, 2026 19:06
Copy link
Copy Markdown

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 simplifies the npm publish GitHub Actions workflow by removing an explicit npm whoami check and eliminating usage of NODE_AUTH_TOKEN, aligning the publish step with npm OIDC/trusted publishing + provenance generation.

Changes:

  • Remove npm whoami from the publish step.
  • Remove NODE_AUTH_TOKEN usage for npm publish, relying on OIDC instead.
Show a summary per file
File Description
.github/workflows/publish.yml Simplifies the npm publish step to use OIDC (no NODE_AUTH_TOKEN) while keeping provenance publishing enabled.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@gracepark gracepark self-assigned this May 5, 2026
@gracepark
Copy link
Copy Markdown
Contributor Author

@TylerJDev - Thank you for the ✅! Would you be able to merge this for me when you have a chance? I currently get:

You're not authorized to push to this branch. Visit https://docs.github.com/repositories/configuring-branches-and-merges-in-your-repository/managing-protected-branches/about-protected-branches for more information.

@TylerJDev
Copy link
Copy Markdown
Contributor

@gracepark, can do! I'll do this for other approved PRs as well.

@TylerJDev TylerJDev merged commit ffc3d69 into main May 6, 2026
8 checks passed
@TylerJDev TylerJDev deleted the gp/migrate-oidc branch May 6, 2026 13:07
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.

3 participants