Skip to content

Replace <LicenseFile> with <LicenseExpression>#1279

Open
ulfllorenz wants to merge 1 commit intopraeclarum:masterfrom
ulfllorenz:license-changes
Open

Replace <LicenseFile> with <LicenseExpression>#1279
ulfllorenz wants to merge 1 commit intopraeclarum:masterfrom
ulfllorenz:license-changes

Conversation

@ulfllorenz
Copy link

Nuget effectively allows only one of the two, but using the expression had two advantages:

  1. It is a tad easier to extract by tooling, because you need not query the github API for a possibly outdated license file.
  2. It is recommended practice by Microsoft for standard licenses https://learn.microsoft.com/en-us/nuget/reference/nuspec#license

This is far from the most urgent improvement. I just thought before I add a line in our internal tooling, I might in parallel spend a miniscule amount of time giving something back.

Nuget effectively allows only one of the two, but using the expression
had two advantages:

1. It is slightly easier to extract by tooling, because you need not
   query the github API for a possibly outdated license file.
2. It is recommended practice by Microsoft for standard licenses
   https://learn.microsoft.com/en-us/nuget/reference/nuspec#license
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.

1 participant