Skip to content

chore: publish to nuget.org via trusted publishing - #1120

Merged
vbreuss merged 1 commit into
mainfrom
chore/nuget-trusted-publishing
Oct 8, 2026
Merged

vbreuss merged 1 commit into
mainfrom
chore/nuget-trusted-publishing

Conversation

@vbreuss

@vbreuss vbreuss commented Oct 8, 2026

Copy link
Copy Markdown
Member

Short-lived OIDC credentials replace the long-lived NuGet API key secret.

Short-lived OIDC credentials replace the long-lived NuGet API key secret.
Copilot AI balanced review requested due to automatic review settings October 8, 2026 19:25
@vbreuss
vbreuss enabled auto-merge (squash) October 8, 2026 19:26

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.

🟢 Approval recommended

The workflow correctly configures and consumes NuGet OIDC credentials in both publishing jobs.

0 open findings

What changed in this PR

Replaces long-lived NuGet API keys with short-lived OIDC credentials for package publishing.

Changes:

  • Grants OIDC token permissions to publishing jobs.
  • Authenticates through NuGet/login@v1.
  • Uses the generated API key for NuGet pushes.
File Description
.github/​workflows/​build.yml Migrates both package publishing jobs to trusted publishing.

🧠 Review effort: Balanced


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

@sonarqubecloud

sonarqubecloud Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
2 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Comment thread .github/workflows/build.yml Dismissed
Comment thread .github/workflows/build.yml Dismissed
@mergify

mergify Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again.

@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown

Test Results

     62 files  ±0       62 suites  ±0   1h 21m 15s ⏱️ -53s
 66 147 tests +1   59 196 ✅ +1   6 951 💤 ±0  0 ❌ ±0 
151 204 runs  ±0  132 065 ✅ ±0  19 139 💤 ±0  0 ❌ ±0 

Results for commit 15535cf. ± Comparison against base commit 70f501b.

This pull request removes 62981 and adds 62981 tests. Note that renamed tests count towards both.
  )
)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(1ca47999-7ae5-4633-9333-73e312b85217)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(1d0224a0-9c22-4442-a15b-8fb10fa064d3)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(26436b92-87be-4882-af77-914b2bd9277a)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(2afa1c1a-ebb9-4554-866c-e82ab14fb20a)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(3099b03c-87e7-4607-b43e-3d412e8d38cc)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(33172175-e8c7-42e0-839f-a0b64ca6d10a)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(36432b24-3368-4cc1-9124-0c0ab51d5da0)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(556f6e96-6742-44b7-b759-d5fe5ebaefdc)
…
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(08ea8af4-a84f-46f2-8e4f-8b7b6785dd5a)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(0bf048b8-efd4-40ad-aaa0-634dc2d13e8c)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(10b058d9-f8ab-4ed6-80db-7e9055d9af11)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(34490066-b847-4a0e-b86d-09ea0ec21e0a)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(498328c9-8c9e-442b-b8fa-45438ddc9a4e)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(5081bc92-f7fc-43a4-8608-4ee316775577)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(52a14075-a8e6-454f-a7d6-a966d9eb2b9c)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(53644226-5579-496b-8dde-374c8c983030)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(5ca2dd69-0655-4980-899c-5fba112ba69d)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(5ff80484-3a34-4c3a-8390-e4c02e0a4084)
…
This pull request removes 6893 skipped tests and adds 6893 skipped tests. Note that renamed tests count towards both.
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(1ca47999-7ae5-4633-9333-73e312b85217)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(1d0224a0-9c22-4442-a15b-8fb10fa064d3)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(26436b92-87be-4882-af77-914b2bd9277a)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(2afa1c1a-ebb9-4554-866c-e82ab14fb20a)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(3099b03c-87e7-4607-b43e-3d412e8d38cc)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(33172175-e8c7-42e0-839f-a0b64ca6d10a)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(36432b24-3368-4cc1-9124-0c0ab51d5da0)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(556f6e96-6742-44b7-b759-d5fe5ebaefdc)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(56a77a20-5af7-4d83-b180-a996cf1d3814)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(690db74e-40b5-4dfc-b204-7f81d67d9968)
…
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(08ea8af4-a84f-46f2-8e4f-8b7b6785dd5a)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(0bf048b8-efd4-40ad-aaa0-634dc2d13e8c)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(10b058d9-f8ab-4ed6-80db-7e9055d9af11)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(34490066-b847-4a0e-b86d-09ea0ec21e0a)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(498328c9-8c9e-442b-b8fa-45438ddc9a4e)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(5081bc92-f7fc-43a4-8608-4ee316775577)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(52a14075-a8e6-454f-a7d6-a966d9eb2b9c)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(53644226-5579-496b-8dde-374c8c983030)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(5ca2dd69-0655-4980-899c-5fba112ba69d)
Testably.Abstractions.AccessControl.Tests.DirectoryInfoAclExtensionsTests ‑ SetAccessControl_ShouldNotUpdateTimes(5ff80484-3a34-4c3a-8390-e4c02e0a4084)
…

@vbreuss
vbreuss merged commit fdd6bc2 into main Oct 8, 2026
17 checks passed
@vbreuss
vbreuss deleted the chore/nuget-trusted-publishing branch October 8, 2026 20:02
@github-actions

Copy link
Copy Markdown

This is addressed in release v7.1.2.

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.

3 participants