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

Include license identifier in the NuGet package #1423

Closed
reduckted opened this issue Dec 3, 2019 · 3 comments
Closed

Include license identifier in the NuGet package #1423

reduckted opened this issue Dec 3, 2019 · 3 comments
Labels

Comments

@reduckted
Copy link

Is your feature request related to a problem? Please describe.
For automated tools that gather the license of NuGet packages, it's really handy for the license to be specified in the NuGet package (specifically in the .nuspec file).

Describe the solution you'd like
Include the license identifier in the NuGet package. I think it would require this in the project file (though I haven't tried this):

<PropertyGroup>
    <PackageLicenseExpression>Apache-2.0 OR MS-PL</PackageLicenseExpression>
</PropertyGroup>

Describe alternatives you've considered
N/A

Additional context
Some links to documentation:

@reduckted
Copy link
Author

Heh, seems like this was added as a comment last year but at some point was removed.

<PackageLicenseUrl>https://raw.githubusercontent.com/JoshClose/CsvHelper/master/LICENSE.txt</PackageLicenseUrl>
<!--<PackageLicenseExpression>MS-PL OR Apache-2.0</PackageLicenseExpression>-->
<PackageRequireLicenseAcceptance>true</PackageRequireLicenseAcceptance>

@JoshClose
Copy link
Owner

Thanks. I'll add it back in. It got removed due to some build stuff, but it needs to go back in. I'm missing the logo and probably some other stuff in there too.

@JoshClose JoshClose added bug and removed feature labels Dec 6, 2019
@JoshClose
Copy link
Owner

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

No branches or pull requests

2 participants