Skip to content

chore: updates workflow to enable npm trusted publishing - #451

Merged
jwndlng merged 2 commits into
mainfrom
jwndlng-patch-1
Oct 27, 2025
Merged

chore: updates workflow to enable npm trusted publishing#451
jwndlng merged 2 commits into
mainfrom
jwndlng-patch-1

Conversation

@jwndlng

@jwndlng jwndlng commented Oct 24, 2025

Copy link
Copy Markdown
Member

NPM package publishing will be migrated to use Trusted Publishing (via OIDC).

To ensure security and restrict access, we will configure the OIDC token request to only be available to jobs running in the dedicated 'release' environment. This environment will be a required condition for obtaining the necessary NPM token.

Changes:

  • Set node version to 24, which includes npm v11 Separate PR.
  • Adds id-token permission required for creating JWT
  • Adds environment release for obtaining the short-lived tokens
  • Pins external action ncipollo/release-action to commit SHA

@jwndlng
jwndlng requested a review from a team as a code owner October 24, 2025 14:10
@jwndlng
jwndlng marked this pull request as draft October 27, 2025 08:44
@jwndlng
jwndlng removed the request for review from a team October 27, 2025 10:15
@jwndlng
jwndlng marked this pull request as ready for review October 27, 2025 10:16
@jwndlng
jwndlng requested a review from a team as a code owner October 27, 2025 10:16
Comment thread .github/workflows/release.yml
Comment thread .github/workflows/release.yml Outdated
Comment thread .node-version
Comment thread .github/workflows/release.yml
@ilbertt

ilbertt commented Oct 27, 2025

Copy link
Copy Markdown
Contributor

Commitizen is checking all the commits in this PR, and it's not happy about 945f17e. Can you try squashing them into one, conventional commits-compliant, commit?

@jwndlng

jwndlng commented Oct 27, 2025

Copy link
Copy Markdown
Member Author

Squashed all commits. Check succeeds now.

@jwndlng
jwndlng enabled auto-merge (squash) October 27, 2025 14:20
@jwndlng
jwndlng disabled auto-merge October 27, 2025 14:20
@jwndlng
jwndlng enabled auto-merge (squash) October 27, 2025 14:24
@jwndlng
jwndlng merged commit a9021cb into main Oct 27, 2025
11 checks passed
@jwndlng
jwndlng deleted the jwndlng-patch-1 branch October 27, 2025 14:26
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