Skip to content

Update packaging to use dotnet pack with centralized MSBuild properties#3908

Merged
Keboo merged 5 commits intomasterfrom
copilot/fix-3907
Aug 31, 2025
Merged

Update packaging to use dotnet pack with centralized MSBuild properties#3908
Keboo merged 5 commits intomasterfrom
copilot/fix-3907

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Aug 29, 2025

  • Migrate from nuget.exe to dotnet pack with MSBuild package properties
  • Create Directory.Build.props with centralized common package properties
  • Update BuildNugets.ps1 to use dotnet pack instead of nuget.exe
  • Remove unnecessary build step from packaging script
  • Refactor common package properties to Directory.Build.props
  • Remove empty Package Content sections from project files

Benefits Achieved:

  • Eliminates external dependency on nuget.exe tool
  • Single source of truth for package metadata in MSBuild properties
  • Better integration with standard .NET CLI and MSBuild tooling
  • Simplified workflow with integrated pack process
  • Automatic dependency resolution from PackageReference and ProjectReference
  • Reduced duplication through centralized Directory.Build.props
  • Clean project files without unnecessary empty sections

Migration Complete:
All packaging has been successfully migrated from .nuspec files to MSBuild properties while maintaining backwards compatibility for package consumers.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Aug 29, 2025

@Keboo 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

Copilot AI and others added 2 commits August 29, 2025 19:28
Co-authored-by: Keboo <952248+Keboo@users.noreply.github.com>
Co-authored-by: Keboo <952248+Keboo@users.noreply.github.com>
Copilot AI changed the title [WIP] Update packaging to use dotnet pack rather than nuget.exe Update packaging to use dotnet pack rather than nuget.exe Aug 29, 2025
Copilot AI requested a review from Keboo August 29, 2025 19:36
@Keboo
Copy link
Copy Markdown
Member

Keboo commented Aug 30, 2025

@copilot review the comments above

…ecessary build step

Co-authored-by: Keboo <952248+Keboo@users.noreply.github.com>
Copilot AI changed the title Update packaging to use dotnet pack rather than nuget.exe Update packaging to use dotnet pack with centralized MSBuild properties Aug 30, 2025
Copilot AI requested a review from Keboo August 30, 2025 04:52
@Keboo
Copy link
Copy Markdown
Member

Keboo commented Aug 31, 2025

@copilot review the above comments

Co-authored-by: Keboo <952248+Keboo@users.noreply.github.com>
Copilot AI requested a review from Keboo August 31, 2025 04:50
@Keboo Keboo marked this pull request as ready for review August 31, 2025 04:53
@Keboo Keboo added this to the 5.3.0 milestone Aug 31, 2025
@Keboo Keboo enabled auto-merge (squash) August 31, 2025 04:59
@Keboo Keboo merged commit b655e99 into master Aug 31, 2025
3 checks passed
@Keboo Keboo deleted the copilot/fix-3907 branch August 31, 2025 05:08
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