-
Notifications
You must be signed in to change notification settings - Fork 966
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
THIRD-PARTY-NOTICES.txt file contains test-only notices #9905
Comments
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Test-only notices should not be included in the main repo's TPN file: https://github.com/dotnet/winforms/blob/main/THIRD-PARTY-NOTICES.TXT
Please remove this notice and include it alongside tests that need it. See this as an example: https://github.com/dotnet/runtime/blob/main/src/tests/JIT/Performance/CodeQuality/Burgers/THIRD-PARTY-NOTICES
This issue was uncovered while updating TPN file for .NET 8 release: dotnet/runtime#91933 The process assembles notices from all repos into a shared TPN file that ships with SDK.
The text was updated successfully, but these errors were encountered: