Skip to content

fix(ci): fix VERSION_BUMP scoping and remove unused NPM token from publish job#1653

Merged
lucasmcdonald3 merged 1 commit intomasterfrom
fix-release-workflow
Mar 20, 2026
Merged

fix(ci): fix VERSION_BUMP scoping and remove unused NPM token from publish job#1653
lucasmcdonald3 merged 1 commit intomasterfrom
fix-release-workflow

Conversation

@lucasmcdonald3
Copy link
Contributor

@lucasmcdonald3 lucasmcdonald3 commented Mar 19, 2026

Issue #, if available:

Description of changes:

Two fixes to the release workflow:

  1. VERSION_BUMP env var was only scoped to the "Configure git" step but referenced in the "Version packages" step. Moved it to the correct step (and removed it from "Configure git" where it was unused) so lerna version actually receives the override when provided.

  2. Removed AWS credentials and NPM token fetch from the publish job. With Lerna 9 and OIDC trusted publishing (id-token: write), the publish auth is handled automatically. The packages are all public so no token is needed for registry reads either.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Check any applicable:

  • Were any files moved? Moving files changes their URL, which breaks all hyperlinks to the files.

@lucasmcdonald3 lucasmcdonald3 requested a review from a team as a code owner March 19, 2026 22:33
…blish job

Move VERSION_BUMP env var to the Version packages step where it is
actually used, and remove it from Configure git where it was unused.
Remove AWS credentials and NPM token fetch from the publish job since
OIDC trusted publishing handles auth via id-token.
@lucasmcdonald3 lucasmcdonald3 merged commit 6fd56ea into master Mar 20, 2026
24 checks passed
@lucasmcdonald3 lucasmcdonald3 deleted the fix-release-workflow branch March 20, 2026 22: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