Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Streamline centalized package version management #1777

Merged
merged 6 commits into from
May 9, 2018

Conversation

lodejard
Copy link
Contributor

@lodejard lodejard commented May 9, 2018

Uses an ItemGroup instead of a PropertyGroup

Assigns Version attributes automatically when a <PackageReference Include="X" /> matches a <PackageManagement Include="X" /> item.

The Version attribute is optional at that point. In this PR they are removed.

You don't need to hand-edit csproj Version attributes anymore. If you add a PackageReference that has the wrong version, then that version will be overridden.

* No reason to have extra dll on disk.
* Also adds a compile-time target to advise when a project is
including an empty build output.
* Adding PackageManagement element automatically overrides
matching PackageReference Version attributes
* The PackageReference element doesn't need to manually
bring in a version property
* Provides a build warning if a project directly references a package
that doesn't have a matching PackageManagement item
Package management didn't quite work for tool references
* The only remaining Version attributes are for System packages
@jtkech
Copy link
Member

jtkech commented May 9, 2018

Thanks a lot for this one 👍

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