WindowsCommunityToolkit icon indicating copy to clipboard operation
WindowsCommunityToolkit copied to clipboard

Centrally manage NuGet dependencies and its versions

Open Nirmal4G opened this issue 2 years ago • 3 comments

Fixes #4180

  • Use Static Graph restore when applicable.
  • Manage Packages Centrally using NuGet's CPVM feature.

Know more about NuGet's Central Package Version Management (CPVM) feature.

PR Type

What kind of change does this PR introduce?

  • Feature
  • Refactoring (no functional changes, no API changes)
  • Build or CI related changes

What is the current behavior?

NuGet package references are spread across project files. It is difficult to track versions and upgrade them properly.

What is the new behavior?

Enables NuGet's upcoming feature to manage package versions centrally using Directory.Packages.props with the following item spec to store a package version.

  <ItemGroup>
    <PackageVersion Include="Vendor.Awesome.Package" Version="11.0.0" />
  </ItemGroup>

PR Checklist

Please check if your PR fulfils the following requirements:

  • [x] Tested code with current supported SDKs
  • [x] New major technical changes in the toolkit have or will be added to the Wiki e.g. build changes, source generators, testing infrastructure, sample creation changes, etc...
  • [x] Header has been added to all new source files (run build/UpdateHeaders.bat)
  • [x] Contains NO breaking changes

Other information

  • If you're editing this patch tree, please rebase on latest HEAD and then commit, without updating from the latest HEAD.
  • When merging, please update the commit title to PR title instead of the default Merge pull request #xxxx from repo/branch, and commit message to either PR message or messages of individual commits. The auto-merge bot does this by default.

Nirmal4G avatar Aug 15 '21 13:08 Nirmal4G

Thanks Nirmal4G for opening a Pull Request! The reviewers will test the PR and highlight if there is any conflict or changes required. If the PR is approved we will proceed to merge the pull request 🙌

ghost avatar Aug 15 '21 13:08 ghost

The build failed, since, there's an issue with MSBuild.Sdk.Extras UWP implicit package reference. Thus, I'll be using a fork of the Extras SDK with fixes for this issue. I'll later patch the Extras SDK itself with those fixes!

Nirmal4G avatar Aug 15 '21 19:08 Nirmal4G

I have extracted unrelated refactoring into its own PR: #4234; I'll rebase this on latest once the PR is merged.

Nirmal4G avatar Sep 09 '21 17:09 Nirmal4G