Skip to content

Conversation

@rsdmike
Copy link
Member

@rsdmike rsdmike commented Jan 8, 2026

PR Checklist

  • Unit Tests have been added for new changes
  • API tests have been updated if applicable
  • All commented code has been removed
  • If you've added a dependency, you've ensured license is compatible with Apache 2.0 and clearly outlined the added dependency.

What are you changing?

Anything the reviewer should know when reviewing this PR?

If the there are associated PRs in other repositories, please link them here (i.e. device-management-toolkit/repo#365 )

Copilot AI review requested due to automatic review settings January 8, 2026 17:51
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 PR updates the CI workflows to use OIDC-based authentication for NPM publishing instead of the traditional token-based approach. The changes remove NODE_AUTH_TOKEN secret references, add id-token permissions, and update both Node.js and semantic-release versions.

Key changes:

  • Adds id-token: write permission for OIDC authentication
  • Updates Node.js version from 22.x to 24.x across both workflows
  • Updates semantic-release from version 24.2.4 to 25.0.2
  • Removes NODE_AUTH_TOKEN environment variable references

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.github/workflows/release.yml Adds OIDC permissions, updates Node.js and semantic-release versions, removes token-based authentication configuration
.github/workflows/node.js.yml Updates Node.js version from 22.x to 24.x

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

@madhavilosetty-intel madhavilosetty-intel merged commit 2b2d16c into main Jan 8, 2026
10 checks passed
@madhavilosetty-intel madhavilosetty-intel deleted the ciUpdates branch January 8, 2026 17:58
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