Skip to content

fix: add id-token: write permission to publish workflow for NuGet OIDC login#89

Merged
ByronMayne merged 2 commits into
masterfrom
copilot/fix-review-comment-88
Jul 9, 2026
Merged

fix: add id-token: write permission to publish workflow for NuGet OIDC login#89
ByronMayne merged 2 commits into
masterfrom
copilot/fix-review-comment-88

Conversation

Copilot AI commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

NuGet/login@v1 uses GitHub OIDC to mint a short-lived API key, but the build job was missing the id-token: write permission — causing the action to fail when attempting to fetch the OIDC token.

Changes

  • .github/workflows/publish.yml — added permissions.id-token: write to the build job:
  build:
    runs-on: windows-latest
    permissions:
      id-token: write   # required for NuGet/login OIDC token exchange

@ByronMayne ByronMayne marked this pull request as ready for review July 8, 2026 02:34
Copilot AI review requested due to automatic review settings July 8, 2026 02:34

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.

Copilot wasn't able to review any files in this pull request.


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

Copilot AI changed the title [WIP] Fix code for review comment on PR 88 fix: add id-token: write permission to publish workflow for NuGet OIDC login Jul 8, 2026
Copilot AI requested a review from ByronMayne July 8, 2026 02:35
@ByronMayne ByronMayne merged commit c29ccf4 into master Jul 9, 2026
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