Skip to content

Conversation

@paulirwin
Copy link
Member

@paulirwin paulirwin commented Oct 22, 2025

Summary

  • Automatically pack NuGet packages on successful builds
  • Upload packages as artifacts for easy download from build details page
  • Only runs on non-PR builds and Windows builds to avoid duplication

Test plan

  • Verify NuGet packages are packed and uploaded on successful main/develop branch builds
  • Verify the step is skipped on pull request builds
  • Verify the step is skipped on Ubuntu and macOS builds
  • Download artifacts from a successful build and verify package contents

🤖 Generated with Claude Code

Automatically pack NuGet packages on successful builds and upload them as artifacts for easy download from the build details page. This only occurs on non-PR builds and Windows builds to avoid duplication.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings October 22, 2025 16:54
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds automated NuGet package creation and artifact uploading to the CI workflow. The changes ensure that NuGet packages are automatically packed on successful builds and made available for download, but only for non-PR builds on Windows to avoid unnecessary duplication across different OS builds.

Key Changes:

  • Added a step to pack NuGet packages using dotnet pack
  • Added a step to upload the packed packages as GitHub Actions artifacts with 30-day retention

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@paulirwin paulirwin merged commit d388485 into main Oct 22, 2025
3 checks passed
@paulirwin paulirwin deleted the chore/nuget-ci branch October 22, 2025 16:57
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.

2 participants