Migrate to Centralized NuGet Package Management (CPM)#81
Conversation
Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This pull request migrates the Onova repository from per-project NuGet package version management to Centralized Package Management (CPM) using a Directory.Packages.props file at the repository root. This approach consolidates all package version declarations in a single location, making it easier to maintain consistent versions across the solution.
Changes:
- Created
Directory.Packages.propswith centralized version management for all 16 unique NuGet packages - Removed
Versionattributes from all<PackageReference>elements across 4 project files while preserving other attributes
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| Directory.Packages.props | New file defining centralized package versions with ManagePackageVersionsCentrally enabled and 16 package version declarations |
| Onova/Onova.csproj | Removed version attributes from 8 package references while preserving PrivateAssets and Condition attributes |
| Onova.Updater/Onova.Updater.csproj | Removed version attributes from 2 package references while preserving PrivateAssets attributes |
| Onova.Tests/Onova.Tests.csproj | Removed version attributes from 9 package references while preserving PrivateAssets attributes |
| Onova.Tests.Dummy/Onova.Tests.Dummy.csproj | Removed version attribute from 1 package reference while preserving PrivateAssets attribute |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
@copilot add the created |
Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #81 +/- ##
=======================================
Coverage 70.10% 70.10%
=======================================
Files 29 29
Lines 572 572
Branches 77 77
=======================================
Hits 401 401
Misses 146 146
Partials 25 25 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Moves all
<PackageReference>versions out of individual.csprojfiles into a singleDirectory.Packages.propsat the repo root, enabling NuGet's Centralized Package Management.Changes
Directory.Packages.props(new) — declares<ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>and 16 alphabetically sorted<PackageVersion>entries covering all packages across the solution.csprojfiles —Version="..."removed from every<PackageReference>; all other attributes (PrivateAssets,Condition, etc.) preservedOnova.sln—Directory.Packages.propsadded to theMiscsolution folder alongside other root files (Directory.Build.props,global.json, etc.)No version conflicts were found — every package used the same version across all projects.
Original prompt
This pull request was created from Copilot chat.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.