Skip to content

Upload consumer-tested NuGet packages from PR builds - #1871

Draft
Jevan Saks (jevansaks) wants to merge 2 commits into
mainfrom
jevansaks-cswin32-pr-package-artifacts
Draft

Jevan Saks (jevansaks) wants to merge 2 commits into
mainfrom
jevansaks-cswin32-pr-package-artifacts

Conversation

@jevansaks

@jevansaks Jevan Saks (jevansaks) commented Nov 1, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Upload the actual Microsoft.Windows.CsWin32.*.nupkg as a dedicated nuget-packages artifact, retained for 14 days.
  • Upload only after the existing integration job restores, builds, runs, and publishes consumers using the packaged analyzer and build task.
  • Fail explicitly if the CsWin32 package is missing or ambiguous, instead of selecting the first arbitrary package.
  • Preserve existing Nerdbank.GitVersioning PR versions. No feed publication.

This PR is based independently on main; it does not include the COM classification changes in #1870. The requested CONTRIBUTING.md addition was removed in 8f46553611447ed670b8987b9981252676634384; the workflow implementation is unchanged.

Validation

Before the documentation-only removal, the Build & Test run completed successfully: build, Windows/Linux/hardware tests, every heavy shard, consumer integration tests, and Validate all passed. Doc validation and CodeQL checks also passed.

The NuGet artifact contains Microsoft.Windows.CsWin32.0.3.0-gc9ed147737.nupkg.

Downloaded this package, restored a standalone net8.0-windows consumer outside the repository using an ordinary PackageReference and a fresh package cache, built with zero warnings/errors, and ran generated GetTickCount successfully. Verified the restored package's SHA256 matches the downloaded nupkg.

Package repository commit: c9ed147737322ef6e392358deff2772b226f08d5 (original PR merge revision).

SHA256: 9A15D49098EE70C64FD2BB6B78813D957DAC3145C17F3F099BCB129FC979E7CE

SHA512: 03CF7351A63EAC4A8EF59D96DCCD023E4160AB50AC76F20E455ADA9EE310A993D137C66EA30C1630923370A67BCEEC821FE73E384C8DA31FC081FD11B7B66D1A

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

This branch has not been deployed

No deployments
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.

1 participant