Skip to content

Embed LICENSE.txt in NuGet packages#14

Merged
DecSmith42 merged 3 commits into
mainfrom
copilot/embed-license-in-nuget
Jul 9, 2026
Merged

Embed LICENSE.txt in NuGet packages#14
DecSmith42 merged 3 commits into
mainfrom
copilot/embed-license-in-nuget

Conversation

Copilot AI commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

The repository's LICENSE.txt was not included in the built NuGet packages, leaving consumers without embedded license information.

Changes

  • Added PackageLicenseFile metadata to the shared Directory.Build.props, following the existing PackageIcon/PackageReadmeFile convention
  • Added LICENSE.txt as a packed None item so it's bundled into every package build (Invex.Rez and Invex.Rez.Configuration)
<PackageLicenseFile>LICENSE.txt</PackageLicenseFile>
...
<None Include="../../LICENSE.txt" Pack="true" PackagePath="\" />

Verified via dotnet pack that both .nupkg outputs now contain LICENSE.txt at the package root, with the .nuspec correctly referencing <license type="file">LICENSE.txt</license>.

Copilot AI requested review from Copilot and removed request for Copilot July 9, 2026 11:56
Copilot AI linked an issue Jul 9, 2026 that may be closed by this pull request
Copilot AI requested review from Copilot and removed request for Copilot July 9, 2026 11:58
Copilot AI changed the title [WIP] Embed license in NuGet package output Embed LICENSE.txt in NuGet packages Jul 9, 2026
Copilot AI requested a review from DecSmith42 July 9, 2026 11:58
@DecSmith42 DecSmith42 marked this pull request as ready for review July 9, 2026 12:00
Copilot AI review requested due to automatic review settings July 9, 2026 12:00
@DecSmith42 DecSmith42 enabled auto-merge July 9, 2026 12:02

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the repository-wide NuGet packing configuration so LICENSE.txt is embedded in all produced NuGet packages, ensuring consumers receive the license file inside the .nupkg and the .nuspec references it via PackageLicenseFile.

Changes:

  • Added PackageLicenseFile metadata to the shared Directory.Build.props.
  • Added LICENSE.txt as a packed None item so it’s included at the package root.

Comment thread Directory.Build.props
Comment thread Directory.Build.props Outdated
@DecSmith42 DecSmith42 disabled auto-merge July 9, 2026 12:14
Copilot AI review requested due to automatic review settings July 9, 2026 12:48

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

@DecSmith42 DecSmith42 enabled auto-merge July 9, 2026 12:52
@DecSmith42 DecSmith42 merged commit e8a3a73 into main Jul 9, 2026
14 checks passed
@DecSmith42 DecSmith42 deleted the copilot/embed-license-in-nuget branch July 9, 2026 13:18
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.

Embed license in nuget

3 participants