Skip to content

Include third-party notices in package-skills NuGet package - #26

Open
kartheekp-ms wants to merge 1 commit into
mainfrom
kartheekp-ms-package-third-party-notices
Open

kartheekp-ms wants to merge 1 commit into
mainfrom
kartheekp-ms-package-third-party-notices

Conversation

@kartheekp-ms

@kartheekp-ms kartheekp-ms commented Oct 24, 2026 •

Copy link
Copy Markdown
Contributor

Fixes: 3083350

Add tool-specific third-party notices under dotnet-package-skills and include THIRD-PARTY-NOTICES.TXT at the NuGet package root. The tool has a dependency on the https://www.nuget.org/packages/YamlDotNet package.

Validated a local pack and confirmed the packaged notices match the source file.

PR build succeeded

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@kartheekp-ms
kartheekp-ms marked this pull request as ready for review October 24, 2026 02:22
@kartheekp-ms
kartheekp-ms requested a review from a team as a code owner October 24, 2026 02:22
@kartheekp-ms kartheekp-ms changed the title Include third-party notices in package-skills Include third-party notices in package-skills NuGet package. Oct 24, 2026
@kartheekp-ms kartheekp-ms changed the title Include third-party notices in package-skills NuGet package. Include third-party notices in package-skills NuGet package Oct 24, 2026

@kartheekp-ms kartheekp-ms Oct 24, 2026 •

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This file content was generated following the guidance provided by the 3083350 work item.

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