Skip to content

fix: Authenticate setup-node against npm.pkg.github.com#780

Merged
ninjasimon merged 1 commit into
mainfrom
fix/preview-workflow-npm-auth
Jun 5, 2026
Merged

fix: Authenticate setup-node against npm.pkg.github.com#780
ninjasimon merged 1 commit into
mainfrom
fix/preview-workflow-npm-auth

Conversation

@ninjasimon
Copy link
Copy Markdown
Contributor

The 'Generate SDK and diff vs main' workflow has been failing on every release PR because actions/setup-node doesn't write authentication config for npm.pkg.github.com when there's no registry-url/scope set. The subsequent npm ci then gets a 401 trying to download @rebilly/regenerator.

This PR adds the same registry-url and scope config that update.yaml already uses so npm ci can pull @rebilly/regenerator from GitHub Packages.

@ninjasimon ninjasimon self-assigned this Jun 5, 2026
@ninjasimon ninjasimon requested a review from a team as a code owner June 5, 2026 17:37
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jun 5, 2026

🦋 Changeset detected

Latest commit: 314b171

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@ninjasimon ninjasimon merged commit c65c66e into main Jun 5, 2026
7 checks passed
@ninjasimon ninjasimon deleted the fix/preview-workflow-npm-auth branch June 5, 2026 19:20
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