FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add NuGet package attestations using GitHub provenance by Copilot · Pull Request #3119 · fluentassertions/fluentassertions · GitHub

/ fluentassertions Public

Add NuGet package attestations using GitHub provenance - #3119

Merged
dennisdoomen merged 2 commits into
mainfrom
copilot/add-nuget-attestations
Nov 8, 2025
ConversationCommits (2)ChecksFiles changed
Merged

Add NuGet package attestations using GitHub provenance#3119
dennisdoomen merged 2 commits into
mainfrom
copilot/add-nuget-attestations

Conversation

Copilot AI commented Nov 6, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Closes #[issue_number]

Adds cryptographically signed provenance attestations to NuGet packages on release, enabling consumers to verify package authenticity and build integrity.

Changes

  • Permissions: Added id-token: write and attestations: write to build job for OIDC and attestation generation
  • Pre-flight check: Verify .nupkg existence before attestation (tag pushes only)
  • Attestation step: Generate provenance using actions/attest-build-provenance@v2 for all packages in Artifacts/

Attestations are created after package build and before upload, only on tag pushes when packages exist.

Original prompt

Add nuget attestations for the nuget pakckage using github's provenance attestations

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI self-assigned this Nov 6, 2025
Copilot AI changed the title [WIP] Add NuGet attestations for the NuGet package Add NuGet package attestations using GitHub provenance Nov 6, 2025
Copilot AI requested a review from dennisdoomen November 6, 2025 18:50

github-actions Bot commented Nov 6, 2025 •
edited
Loading

Copy link
Copy Markdown

Test Results

    37 files  ±0      37 suites  ±0   2m 55s ⏱️ +18s
 6 267 tests ±0   6 267 ✅ ±0  0 💤 ±0  0 ❌ ±0 
38 948 runs  ±0  38 948 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 518a9fc. ± Comparison against base commit 2ff081c.

This pull request removes 10 and adds 8 tests. Note that renamed tests count towards both. 
FluentAssertions.Specs.Streams.StreamAssertionSpecs+HaveLength ‑ When_a_throwing_stream_should_have_a_length_it_should_fail(exception: System.ObjectDisposedException: Cannot access a disposed object.
FluentAssertions.Specs.Streams.StreamAssertionSpecs+HaveLength ‑ When_a_throwing_stream_should_have_a_length_it_should_fail(exception: System.ObjectDisposedException: Cannot access a disposed object.
FluentAssertions.Specs.Streams.StreamAssertionSpecs+HavePosition ‑ When_a_throwing_stream_should_have_a_position_it_should_fail(exception: System.ObjectDisposedException: Cannot access a disposed object.
FluentAssertions.Specs.Streams.StreamAssertionSpecs+HavePosition ‑ When_a_throwing_stream_should_have_a_position_it_should_fail(exception: System.ObjectDisposedException: Cannot access a disposed object.
FluentAssertions.Specs.Streams.StreamAssertionSpecs+NotHaveLength ‑ When_a_throwing_stream_should_not_have_a_length_it_should_fail(exception: System.ObjectDisposedException: Cannot access a disposed object.
FluentAssertions.Specs.Streams.StreamAssertionSpecs+NotHaveLength ‑ When_a_throwing_stream_should_not_have_a_length_it_should_fail(exception: System.ObjectDisposedException: Cannot access a disposed object.
FluentAssertions.Specs.Streams.StreamAssertionSpecs+NotHavePosition ‑ When_a_throwing_stream_should_not_have_a_position_it_should_fail(exception: System.ObjectDisposedException: Cannot access a disposed object.
FluentAssertions.Specs.Streams.StreamAssertionSpecs+NotHavePosition ‑ When_a_throwing_stream_should_not_have_a_position_it_should_fail(exception: System.ObjectDisposedException: Cannot access a disposed object.
Object name: 'GetLengthExceptionMessage'.)
Object name: 'GetPositionExceptionMessage'.)

FluentAssertions.Specs.Streams.StreamAssertionSpecs+HaveLength ‑ When_a_throwing_stream_should_have_a_length_it_should_fail(exception: System.ObjectDisposedException: Cannot access a disposed object.
Object name: 'GetLengthExceptionMessage'.)
FluentAssertions.Specs.Streams.StreamAssertionSpecs+HaveLength ‑ When_a_throwing_stream_should_have_a_length_it_should_fail(exception: System.ObjectDisposedException: Cannot access a disposed object.
Object name: 'GetLengthExceptionMessage'.)
FluentAssertions.Specs.Streams.StreamAssertionSpecs+HavePosition ‑ When_a_throwing_stream_should_have_a_position_it_should_fail(exception: System.ObjectDisposedException: Cannot access a disposed object.
Object name: 'GetPositionExceptionMessage'.)
FluentAssertions.Specs.Streams.StreamAssertionSpecs+HavePosition ‑ When_a_throwing_stream_should_have_a_position_it_should_fail(exception: System.ObjectDisposedException: Cannot access a disposed object.
Object name: 'GetPositionExceptionMessage'.)
FluentAssertions.Specs.Streams.StreamAssertionSpecs+NotHaveLength ‑ When_a_throwing_stream_should_not_have_a_length_it_should_fail(exception: System.ObjectDisposedException: Cannot access a disposed object.
Object name: 'GetLengthExceptionMessage'.)
FluentAssertions.Specs.Streams.StreamAssertionSpecs+NotHaveLength ‑ When_a_throwing_stream_should_not_have_a_length_it_should_fail(exception: System.ObjectDisposedException: Cannot access a disposed object.
Object name: 'GetLengthExceptionMessage'.)
FluentAssertions.Specs.Streams.StreamAssertionSpecs+NotHavePosition ‑ When_a_throwing_stream_should_not_have_a_position_it_should_fail(exception: System.ObjectDisposedException: Cannot access a disposed object.
Object name: 'GetPositionExceptionMessage'.)
FluentAssertions.Specs.Streams.StreamAssertionSpecs+NotHavePosition ‑ When_a_throwing_stream_should_not_have_a_position_it_should_fail(exception: System.ObjectDisposedException: Cannot access a disposed object.
Object name: 'GetPositionExceptionMessage'.)

♻️ This comment has been updated with latest results.

coveralls commented Nov 6, 2025 •
edited
Loading

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 19192192660

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 97.154%
Totals
Change from base Build 18872999242: 0.0%
Covered Lines: 12826
Relevant Lines: 13046
💛 - Coveralls

github-actions Bot commented Nov 6, 2025 •
edited
Loading

Copy link
Copy Markdown

Qodana for .NET

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

dennisdoomen force-pushed the copilot/add-nuget-attestations branch from 66b076d to 34b4022 Compare November 8, 2025 07:29
dennisdoomen added the building Building and Infrastructure of Fluent Assertions label Nov 8, 2025
dennisdoomen requested a review from jnyrup November 8, 2025 07:29
jnyrup reviewed Nov 8, 2025
View reviewed changes

jnyrup left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I've got no experience with attestation but it seems like a good thing to do.
What triggered you to enable this?
I only read this blog post so far.
https://andrewlock.net/creating-provenance-attestations-for-nuget-packages-in-github-actions/

Comment thread .github/workflows/build.yml

dennisdoomen commented Nov 8, 2025

Copy link
Copy Markdown
Member

What triggered you to enable this?

This suggestion was the trigger dennisdoomen/dotnet-library-starter-kit#35

Add NuGet package attestations using GitHub provenance
518a9fc 
Co-authored-by: dennisdoomen <572734+dennisdoomen@users.noreply.github.com>
dennisdoomen force-pushed the copilot/add-nuget-attestations branch from 34b4022 to 518a9fc Compare November 8, 2025 11:07
dennisdoomen marked this pull request as ready for review November 8, 2025 11:07
dennisdoomen merged commit 889bc88 into main Nov 8, 2025
14 checks passed
dennisdoomen deleted the copilot/add-nuget-attestations branch November 8, 2025 19:05
This was referenced Mar 16, 2026
Merged
Merged
Merged
Closed
Closed
Merged
Open
Merged
This was referenced Aug 22, 2026
Merged
Closed
Open
Open
Open
Open
Closed
Open
Open
Open
Open
Open
Open
Open
Closed
Closed
Closed
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Merged
dependabot Bot mentioned this pull request Aug 29, 2026
Open
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

dennisdoomen dennisdoomen approved these changes
jnyrup jnyrup approved these changes

Assignees

dennisdoomen

Copilot code review Copilot

Labels

building Building and Infrastructure of Fluent Assertions

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

Back | FazBrowse Home | New Git URL