Skip to content

ci: add build-reproducibility verification (#135) - #218

Merged
Chris-Wolfgang merged 2 commits into
vNextfrom
ci/reproducible-build
Jul 21, 2026
Merged

ci: add build-reproducibility verification (#135)#218
Chris-Wolfgang merged 2 commits into
vNextfrom
ci/reproducible-build

Conversation

@Chris-Wolfgang

Copy link
Copy Markdown
Owner

Stacked on #217. Adds reproducible-build.yaml — checks the commit out to two independent directories, builds both TestKit assemblies with -p:ContinuousIntegrationBuild=true, and fails if the .dll hashes differ. Proves path-independent reproducibility. Adds REPRODUCIBLE-BUILD.md. Ported from D20-Dice, extended to both packable projects.

Scope note

  • ✅ Same-runner path-independent byte-identity for both assemblies + doc.
  • ⏳ Cross-OS byte-identity (ubuntu vs windows) deferred — .pdb/metadata can differ across SDK/OS; tracked as follow-up in REPRODUCIBLE-BUILD.md.
  • CI is the definitive verification (faithful port of D20's green pattern, same deterministic Directory.Build.props, no source generator).

Closes #135 when the vNext cycle merges to main.

🤖 Generated with Claude Code

Copilot AI review requested due to automatic review settings July 21, 2026 19:35

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Comment thread .github/workflows/reproducible-build.yaml Fixed
Comment thread .github/workflows/reproducible-build.yaml Fixed
Chris-Wolfgang and others added 2 commits July 21, 2026 16:30
Adds reproducible-build.yaml — checks the commit out to two directories, builds
both TestKit assemblies with ContinuousIntegrationBuild=true, and fails if the
.dll hashes differ (path-independent reproducibility). Adds REPRODUCIBLE-BUILD.md
documenting the guarantee + third-party verification steps. Ported from D20-Dice,
extended to both packable projects.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…d.yaml (#143)

Aligns with the fleet SHA-pin migration (#219): checkout v7.0.0 -> v7,
setup-dotnet v5 -> v6, upload-artifact v4 -> v7, upload-sarif v3/v4 -> v4.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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