Migrate to Centralized NuGet Package Management (CPM)#416
Conversation
Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>
|
@copilot add the created Also, formatting fails |
There was a problem hiding this comment.
Pull request overview
This pull request migrates the LightBulb repository from per-project package version management to NuGet's Centralized Package Management (CPM). All package versions are consolidated into a new Directory.Packages.props file at the repository root, while individual project files retain their PackageReference elements without version attributes.
Changes:
- Created
Directory.Packages.propswith 19 centralized package versions - Removed
Versionattributes from allPackageReferenceelements across 4 project files - Preserved all other attributes (
PrivateAssets,Condition) on package references
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| Directory.Packages.props | New file defining centralized package versions with ManagePackageVersionsCentrally enabled and 19 PackageVersion entries in alphabetical order |
| LightBulb/LightBulb.csproj | Removed Version attributes from 12 PackageReference elements while preserving PrivateAssets and Condition attributes |
| LightBulb.PlatformInterop/LightBulb.PlatformInterop.csproj | Removed Version attribute from CSharpier.MsBuild PackageReference while preserving PrivateAssets |
| LightBulb.Core/LightBulb.Core.csproj | Removed Version attributes from 2 PackageReference elements while preserving PrivateAssets |
| LightBulb.Core.Tests/LightBulb.Core.Tests.csproj | Removed Version attributes from 7 PackageReference elements while preserving PrivateAssets |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>
Done in b35b83a:
Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #416 +/- ##
=======================================
Coverage 96.35% 96.35%
=======================================
Files 7 7
Lines 192 192
Branches 15 15
=======================================
Hits 185 185
Misses 6 6
Partials 1 1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Directory.Packages.propswith all package versions (alphabetically sorted)LightBulb/LightBulb.csprojto removeVersionattributesLightBulb.Core/LightBulb.Core.csprojto removeVersionattributesLightBulb.Core.Tests/LightBulb.Core.Tests.csprojto removeVersionattributesLightBulb.PlatformInterop/LightBulb.PlatformInterop.csprojto removeVersionattributesDirectory.Packages.propsto theMiscsolution folder inLightBulb.slnAvalonia.DiagnosticsPackageReference to single line (only 2 attrs remain after removing Version)Original prompt
This pull request was created from Copilot chat.
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.