Skip to content

ci: centralize GitHub Packages npm auth setup#34345

Open
mpreyskurantov wants to merge 2 commits into
mainfrom
26_2-34326
Open

ci: centralize GitHub Packages npm auth setup#34345
mpreyskurantov wants to merge 2 commits into
mainfrom
26_2-34326

Conversation

@mpreyskurantov

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 centralizes GitHub Packages npm authentication setup in CI workflows, so SBOM generation and package publishing no longer need to inline the auth configuration in their command blocks.

Changes:

  • Adds a dedicated “Set GitHub Packages auth” step to workflows that build SBOMs and publish packages.
  • Simplifies SBOM build steps by removing the inline auth setup and running only pnpm nx build sbom.
  • Renames the publishing step label to “Publish to GitHub Packages” for clarity.

Reviewed changes

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

File Description
.github/workflows/packages_publishing.yml Extracts GitHub Packages auth setup into its own step and simplifies SBOM + publish steps.
.github/workflows/build_all.yml Extracts GitHub Packages auth setup into its own step and simplifies the SBOM build command.

Comment thread .github/workflows/build_all.yml
Copilot AI review requested due to automatic review settings July 16, 2026 14:24

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

Comment thread .github/workflows/packages_publishing.yml
Comment thread .github/workflows/packages_publishing.yml
Comment thread .github/workflows/build_all.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants