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

Enable GitVersionTask to work with NuGet v3 #563

Merged
merged 2 commits into from
Aug 12, 2015

Conversation

clairernovotny
Copy link

NuGet v3 does not allow .props/.targets files directly in the \build directory. They must be in a subdirectory.

This PR puts the targets file in \Build\portable-net+sl+win+wpa+wp and \lib\dotnet so it'll work with any NuGet v2 or v3 client. The relative paths in the .targets file has been adjusted to match the new location.

Without this, the targets are never included in UWP or other project.json based projects (like modern PCL, etc).

@gep13
Copy link
Member

gep13 commented Aug 11, 2015

@SimonCropp is this similar to the changes that had to be made in Fody?

@clairernovotny
Copy link
Author

@SimonCropp
Copy link
Contributor

@gep13 yes. @onovotny this is very good work

@JakeGinnivan this is good for a merge from my perspective

JakeGinnivan added a commit that referenced this pull request Aug 12, 2015
Enable GitVersionTask to work with NuGet v3
@JakeGinnivan JakeGinnivan merged commit a759194 into GitTools:master Aug 12, 2015
@clairernovotny clairernovotny deleted the nuget3 branch August 12, 2015 12:08
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.

None yet

4 participants