Skip to content

Conversation

@carldebilly
Copy link
Owner

@carldebilly carldebilly commented Sep 6, 2025

This PR configures the project for symbol package publishing to NuGet by adding support for .snupkg symbol packages and optimizing the CI build configuration.

  • Enables symbol package creation and publishing with proper source embedding
  • Updates CI pipeline to run on Ubuntu and handle both .nupkg and .snupkg files
  • Optimizes build configuration by disabling unnecessary cloud build features

@carldebilly carldebilly requested a review from Copilot September 6, 2025 05:19
@carldebilly carldebilly marked this pull request as ready for review September 6, 2025 05:20
Copy link

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 configures the project for symbol package publishing to NuGet by adding support for .snupkg symbol packages and optimizing the CI build configuration.

  • Enables symbol package creation and publishing with proper source embedding
  • Updates CI pipeline to run on Ubuntu and handle both .nupkg and .snupkg files
  • Optimizes build configuration by disabling unnecessary cloud build features

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
version.json Disables cloud build variables and build numbering for optimized CI
Directory.Build.props Adds source embedding and CI build detection for symbol packages
.github/workflows/build.yml Updates runner OS and adds symbol package handling to build and publish steps

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

@carldebilly carldebilly merged commit 49ef9f6 into master Sep 6, 2025
2 checks passed
@carldebilly carldebilly deleted the dev/packaging branch September 6, 2025 05:21
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