Skip to content
This repository was archived by the owner on Apr 16, 2020. It is now read-only.

Conversation

@paulomorgado
Copy link
Contributor

Added True to build NuGet packages with the developmentDependency metadata

@ctaggart
Copy link
Owner

ctaggart commented Jun 7, 2018

@paulomorgado I thought you did that already as #331.

@paulomorgado
Copy link
Contributor Author

For some reason, that one seems to have escaped.

@ctaggart
Copy link
Owner

ctaggart commented Jun 7, 2018

<!-- https://github.com/NuGet/Home/wiki/Adding-nuget-pack-as-a-msbuild-target -->
<IncludeBuildOutput>false</IncludeBuildOutput>
<!--<CopyLocalLockFileAssemblies>true</CopyLocalLockFileAssemblies>-->
<DevelopmentDependency>True</DevelopmentDependency>
Copy link
Owner

Choose a reason for hiding this comment

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

Added <DevelopmentDependency>True</DevelopmentDependency> to build NuGet packages with the developmentDependency metadata
@paulomorgado paulomorgado force-pushed the DevelopmentDependency branch from 6c877ab to e1097f8 Compare June 7, 2018 23:23
@paulomorgado
Copy link
Contributor Author

Now that you mention it, I did had the impression that I had already sent a PR for this.

But when I checked I checked https://www.nuget.org/packages/SourceLink.Copy.PdbFiles/2.8.1 and saw it was missing it, I thought I had forgotten.

@ctaggart ctaggart changed the title Development dependency Development dependency for SourceLink.Copy.PdbFiles Jun 8, 2018
@ctaggart ctaggart added this to the 2.8.2 milestone Jun 8, 2018
@ctaggart ctaggart merged commit fcf7ff4 into ctaggart:master Jun 8, 2018
@ctaggart
Copy link
Owner

ctaggart commented Jun 8, 2018

2.8.2 published. Thanks!

@paulomorgado
Copy link
Contributor Author

No! Thanks for providing the tools!

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.

2 participants