Migrate to Centralized NuGet Package Management (CPM)#57
Conversation
Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>
|
@copilot formatting issues |
There was a problem hiding this comment.
Pull request overview
This PR migrates the repository from per-project NuGet package version management to Centralized Package Management (CPM), consolidating all package versions into a single Directory.Packages.props file at the repository root. This follows Microsoft's recommended approach for managing package versions across multi-project solutions and improves maintainability by eliminating version duplication.
Changes:
- Created
Directory.Packages.propswith centralized version definitions for 7 NuGet packages - Removed
Versionattributes from<PackageReference>elements in all 6 project files while preserving other attributes - Added
Directory.Packages.propsto the solution's "Misc" folder for visibility
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| Directory.Packages.props | New file enabling CPM with alphabetically sorted PackageVersion entries for all 7 packages |
| DotnetRuntimeBootstrapper/DotnetRuntimeBootstrapper.csproj | Removed Version attributes from 6 PackageReference elements, preserved all other attributes |
| DotnetRuntimeBootstrapper.sln | Added Directory.Packages.props to the Misc solution folder |
| DotnetRuntimeBootstrapper.Demo.Gui/DotnetRuntimeBootstrapper.Demo.Gui.csproj | Removed Version attribute from CSharpier.MsBuild PackageReference |
| DotnetRuntimeBootstrapper.Demo.Cli/DotnetRuntimeBootstrapper.Demo.Cli.csproj | Removed Version attribute from CSharpier.MsBuild PackageReference |
| DotnetRuntimeBootstrapper.AppHost.Gui/DotnetRuntimeBootstrapper.AppHost.Gui.csproj | Removed Version attributes from 3 PackageReference elements |
| DotnetRuntimeBootstrapper.AppHost.Core/DotnetRuntimeBootstrapper.AppHost.Core.csproj | Removed Version attributes from 3 PackageReference elements |
| DotnetRuntimeBootstrapper.AppHost.Cli/DotnetRuntimeBootstrapper.AppHost.Cli.csproj | Removed Version attributes from 3 PackageReference elements |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…rops style Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>
|
@copilot run |
Build passes — all non-Windows projects restore and compile successfully with the CPM changes. The only failure is the pre-existing |
Directory.Packages.propswith all package versions (alphabetically sorted).csprojfiles to removeVersion="..."from<PackageReference>elementsDirectory.Packages.propsto the.slnfile under the existing "Misc" solution items folderDirectory.Packages.propsto matchDirectory.Build.propsstyle (blank lines after<Project>and before</Project>)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.