Add transitive FrameworkReference guidance to NU1510 docs#3575
Conversation
|
Learn Build status updates of commit 1e70f84: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 93775ad: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 42c65bf: ✅ Validation status: passed
For more details, please refer to the build report. |
There was a problem hiding this comment.
Pull request overview
This PR updates the NU1510 warning documentation to cover a transitive FrameworkReference scenario where pruning guidance requires the consumer project to declare the framework reference directly.
Changes:
- Adds
ai-usagemetadata and updates the document date. - Renames the original content to Scenario 1.
- Adds Scenario 2 with
Library.csprojandConsumer.csprojexamples for directFrameworkReferenceguidance.
NU1510 documentation did not cover the case where pruning depends on a framework reference that arrives through a project reference. This left a gap for consumers who need to add the framework reference directly before NuGet can recognize an otherwise unnecessary package reference.
Problem clarification
Microsoft.AspNetCore.Appand a consumer that references the library.FrameworkReferenceas sufficient for pruning guidance.Guidance added
FrameworkReferencedirectly.PackageReference.NU1510 example expansion
NU1510.mdwithLibrary.csprojandConsumer.csproj.FrameworkReferencerequirement in this scenario.