Skip to content

chore: adopt Central Package Management for solution dependencies#196

Merged
NichUK merged 2 commits into
mainfrom
copilot/repo-assist-chore-introduce-cpm
Jun 18, 2026
Merged

chore: adopt Central Package Management for solution dependencies#196
NichUK merged 2 commits into
mainfrom
copilot/repo-assist-chore-introduce-cpm

Conversation

Copilot AI commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

This change introduces Central Package Management (CPM) at the solution root so NuGet versions are defined once instead of being duplicated across project files. Package version pins are now centralized for test and benchmark dependencies.

  • What changed

    • Added Directory.Packages.props with:
      • ManagePackageVersionsCentrally=true
      • Centralized versions for all solution package dependencies (8 total)
    • Updated tests/FixedPointNano.Tests/FixedPointNano.Tests.csproj:
      • Removed inline Version attributes from all PackageReference entries
      • Kept existing IncludeAssets / PrivateAssets settings unchanged
    • Updated benchmarks/FixedPointNano.Benchmarks/FixedPointNano.Benchmarks.csproj:
      • Removed inline Version from BenchmarkDotNet reference
  • Central version manifest

    <Project>
      <PropertyGroup>
        <ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
      </PropertyGroup>
      <ItemGroup>
        <PackageVersion Include="BenchmarkDotNet" Version="0.14.0" />
        <PackageVersion Include="coverlet.collector" Version="6.0.4" />
        <PackageVersion Include="coverlet.msbuild" Version="8.0.1" />
        <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="17.14.0" />
        <PackageVersion Include="Moq" Version="4.20.72" />
        <PackageVersion Include="NUnit" Version="4.3.2" />
        <PackageVersion Include="NUnit.Analyzers" Version="4.7.0" />
        <PackageVersion Include="NUnit3TestAdapter" Version="5.0.0" />
      </ItemGroup>
    </Project>

Copilot AI changed the title [WIP] Introduce Central Package Management (CPM) chore: adopt Central Package Management for solution dependencies Jun 18, 2026
Copilot AI requested a review from NichUK June 18, 2026 19:05
@NichUK NichUK marked this pull request as ready for review June 18, 2026 19:05
Copilot AI review requested due to automatic review settings June 18, 2026 19:05
@NichUK NichUK merged commit ec1daed into main Jun 18, 2026
1 check passed
@NichUK NichUK deleted the copilot/repo-assist-chore-introduce-cpm branch June 18, 2026 19:05

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adopts NuGet Central Package Management (CPM) so package versions are defined once at the repository root instead of being duplicated across individual project files, centralizing test and benchmark dependency version pins.

Changes:

  • Added Directory.Packages.props enabling ManagePackageVersionsCentrally and defining 8 centralized PackageVersion entries.
  • Removed inline Version="..." attributes from PackageReference items in the test project.
  • Removed the inline Version="..." attribute from the BenchmarkDotNet reference in the benchmarks project.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
Directory.Packages.props Introduces CPM and centrally pins all solution NuGet package versions.
tests/FixedPointNano.Tests/FixedPointNano.Tests.csproj Switches test package references to rely on centrally managed versions.
benchmarks/FixedPointNano.Benchmarks/FixedPointNano.Benchmarks.csproj Switches BenchmarkDotNet reference to rely on centrally managed version.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

3 participants