Skip to content
This repository has been archived by the owner on Jan 12, 2024. It is now read-only.

Added baseNuspecFile support and removed duplicate files exception #149

Closed
wants to merge 3 commits into from

Conversation

thorgeir
Copy link

@thorgeir thorgeir commented Nov 3, 2017

Added support to base the nuget package of an existing nuspec file. This was possible in the NuProj projects and I've duplicated the behavior.

Also I've removed the duplicate file exception. The _PackageContent variable will contain duplicate file references when one or more project references the same files/assemblies, this should not cause an exception. Instead the duplicates are removed and the first reference is used.

@dnfclas
Copy link

dnfclas commented Nov 3, 2017

@thorgeir,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.

It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

@kzu
Copy link
Contributor

kzu commented Nov 3, 2017

Could you please split the two in separate PRs for clarity and tracking purposes? That will make it easier to also discuss each one separately.

Thanks!

@dnfclas
Copy link

dnfclas commented Nov 3, 2017

@thorgeir, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.

Thanks, .NET Foundation Pull Request Bot

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants