Skip to content

Update Node.js version and permissions in publish.yml#37

Merged
gracepark merged 5 commits into
masterfrom
gp/oidc-migration
May 14, 2026
Merged

Update Node.js version and permissions in publish.yml#37
gracepark merged 5 commits into
masterfrom
gp/oidc-migration

Conversation

@gracepark
Copy link
Copy Markdown
Contributor

Copilot AI review requested due to automatic review settings May 8, 2026 19:11
@gracepark gracepark requested a review from a team as a code owner May 8, 2026 19:11
@gracepark gracepark requested a review from bteng22 May 8, 2026 19:11
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

Updates the release publishing workflow to support the OIDC/provenance-based publishing flow and modernize the Node.js runtime used during publishes.

Changes:

  • Add explicit job permissions (including id-token: write) to enable OIDC-based provenance.
  • Upgrade the publish jobs to Node.js 20.
  • Switch publish commands to use npm publish --provenance and remove long-lived npm token usage from the workflow.
Show a summary per file
File Description
.github/workflows/publish.yml Updates Node version, job permissions, and publish steps to support OIDC/provenance publishing.

Copilot's findings

Tip

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

Comments suppressed due to low confidence (1)

.github/workflows/publish.yml:36

  • Same as above: actions/checkout@v3 / actions/setup-node@v3 are on an older GitHub Actions runtime; bumping to v4 reduces the risk of the publish workflow breaking due to action runtime deprecations.
      - uses: actions/checkout@v3
      - uses: actions/setup-node@v3
        with:
  • Files reviewed: 1/1 changed files
  • Comments generated: 2

Comment thread .github/workflows/publish.yml
Comment thread .github/workflows/publish.yml Outdated
Comment thread .github/workflows/publish.yml Outdated
- uses: actions/setup-node@v5
with:
node-version: 14
node-version: 20
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Node version in other PRs is 24 but it's 20 here, is that expected?

@gracepark gracepark merged commit 4de6e65 into master May 14, 2026
4 checks passed
@gracepark gracepark deleted the gp/oidc-migration branch May 14, 2026 19:18
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