Skip to content

Enable nuget Trusted Publishing - #206

Merged
pdeffendol merged 2 commits into
mainfrom
trusted-publish
Sep 2, 2026
Merged

pdeffendol merged 2 commits into
mainfrom
trusted-publish

Conversation

@pdeffendol

@pdeffendol pdeffendol commented Sep 2, 2026 •

Copy link
Copy Markdown
Contributor

This replaces the explicit API key publish method. We must define a secret in the repo NUGET_USER which is the GitHub username of the person who has set up a trusted publishing configuration.

What type of PR is this? (check all applicable)

  • 🍕 Feature
  • 🐛 Bug Fix
  • 📝 Documentation
  • 🎨 Style
  • 🧑‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Tests
  • 🤖 Build/CI
  • 📦 Chore (Version bump, release, etc.)
  • ⏩ Revert

This replaces the explicit API key publish method.  We must define a
secret in the repo NUGET_USER which is the GitHub username of the person
who has set up a trusted publishing configuration.

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.

🔵 Needs a closer look

It changes the production release/publishing path to nuget.org and requires validating external configuration (trusted publishing + secrets) that can’t be verified from the diff alone.

Pull request overview

Updates the release workflow to publish NuGet packages via NuGet.org Trusted Publishing (OIDC) instead of using a long-lived NuGet API key, aligning the repo’s release process with modern, token-based publishing.

Changes:

  • Grants id-token: write permissions so the workflow can request an OIDC token for NuGet trusted publishing.
  • Adds a NuGet/login@v1 step to obtain a short-lived NuGet API key for stable releases.
  • Switches the nuget.org push step to use the temporary API key output from the login step.
File summaries
File Description
.github/workflows/release.yml Adds OIDC permission + NuGet/login step and uses its temporary API key to push stable packages to nuget.org.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/release.yml Resolved
@pdeffendol
pdeffendol merged commit ba70e01 into main Sep 2, 2026
1 check passed
@pdeffendol
pdeffendol deleted the trusted-publish branch September 2, 2026 22:24
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