Migrate to Centralized NuGet Package Management (CPM)#23
Conversation
Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>
|
@copilot formatting issues |
|
@copilot formatting issues |
There was a problem hiding this comment.
Pull request overview
This pull request migrates the Gress repository to use NuGet's Centralized Package Management (CPM), consolidating all package version definitions from individual .csproj files into a single Directory.Packages.props file at the repository root. This enables centralized version management across all projects in the solution, improving maintainability and consistency.
Changes:
- Created
Directory.Packages.propswith all 15 package versions defined centrally and sorted alphabetically - Removed
Versionattributes from all<PackageReference>elements in project files while preserving other attributes - Added
Directory.Packages.propsto the solution's Misc folder for visibility
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>true</ManagePackageVersionsCentrally> and 15 <PackageVersion> entries |
| Gress/Gress.csproj | Removed version attributes from 3 package references (CSharpier.MsBuild, Microsoft.SourceLink.GitHub, PolyShim), preserved PrivateAssets attributes |
| Gress.Tests/Gress.Tests.csproj | Removed version attributes from 7 package references, preserved PrivateAssets attributes |
| Gress.Demo/Gress.Demo.csproj | Removed version attributes from 7 package references including conditional Avalonia.Diagnostics, preserved PrivateAssets and Condition attributes |
| Gress.sln | Added Directory.Packages.props to the Misc solution items folder |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>
…ving Version Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>
Fixed in 1302ea4 — collapsed the multi-line 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 #23 +/- ##
=======================================
Coverage 90.52% 90.52%
=======================================
Files 11 11
Lines 190 190
Branches 19 19
=======================================
Hits 172 172
Misses 13 13
Partials 5 5 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Directory.Packages.propsat repo root with all unique packages (sorted alphabetically)Version="..."from all<PackageReference>elements in the 3.csprojfilesDirectory.Packages.propsto theMiscSolution Items folder inGress.slnDirectory.Packages.propsto matchDirectory.Build.propsstyle (blank lines after<Project>, between groups, and before</Project>)Avalonia.DiagnosticsPackageReferenceto single line now thatVersionattribute is removedOriginal prompt
This pull request was created from Copilot chat.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.