Skip to content

Use NuGet trusted publishing - #15

Merged
ericstj merged 1 commit into
mainfrom
trusted-publishing
Jul 23, 2026
Merged

Use NuGet trusted publishing#15
ericstj merged 1 commit into
mainfrom
trusted-publishing

Conversation

@ericstj

@ericstj ericstj commented Jul 23, 2026

Copy link
Copy Markdown
Owner

Summary

  • authenticate to NuGet.org with GitHub OIDC
  • isolate tag publishing in a least-privilege job
  • remove the long-lived NuGet API key dependency

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: a9621082-f9c8-4ec8-ac3c-faaf34eb7d0f
Copilot AI review requested due to automatic review settings July 23, 2026 01:30

Copilot AI left a comment

Copy link
Copy Markdown

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 workflow to publish NuGet packages using GitHub OIDC (NuGet Trusted Publishing) on version tags, removing reliance on a long-lived NuGet API key and isolating publishing into a least-privilege job.

Changes:

  • Adds a dedicated publish job gated to refs/tags/v* that depends on the build job artifacts.
  • Uses NuGet/login@v1 to obtain a short-lived NuGet API key via OIDC for dotnet nuget push.
Show a summary per file
File Description
.github/workflows/ci.yml Adds a tag-only publish job using OIDC-based NuGet authentication and artifact-based package publishing.

Review details

Tip

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

  • Files reviewed: 1/1 changed files
  • Comments generated: 2
  • Review effort level: Low

Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml
@ericstj
ericstj requested a review from Copilot July 23, 2026 01:51
@ericstj
ericstj merged commit 1808499 into main Jul 23, 2026
4 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review details

  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Low

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