Conversation
|
Learn Build status updates of commit b24ad7b:
|
Agent-Logs-Url: https://github.com/NuGet/docs.microsoft.com-nuget/sessions/939ab205-9284-4871-b331-25cae2734b0a Co-authored-by: nkolev92 <2878341+nkolev92@users.noreply.github.com>
|
Learn Build status updates of commit ee8e7b1: ✅ 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 |
Agent-Logs-Url: https://github.com/NuGet/docs.microsoft.com-nuget/sessions/9b70f715-3e55-48e8-8ca6-24670c205864 Co-authored-by: nkolev92 <2878341+nkolev92@users.noreply.github.com>
|
Learn Build status updates of commit 95ed256: ✅ 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 |
There was a problem hiding this comment.
Pull request overview
This PR adds documentation for the newly introduced NU1019 error/warning (invalid TargetFramework / TargetFrameworks values containing path separators or non-ASCII characters) and links it from the Errors and Warnings reference index.
Changes:
- Added a new reference page describing NU1019 scenarios, analysis-level gating, and remediation guidance.
- Updated the main Errors and Warnings index to include NU1019 under “Invalid input errors”.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| docs/reference/errors-and-warnings/NU1019.md | New NU1019 documentation page describing the scenario, gating behavior, and suggested fixes. |
| docs/reference/Errors-and-Warnings.md | Adds NU1019 to the invalid input errors list so it’s discoverable from the index. |
Agent-Logs-Url: https://github.com/NuGet/docs.microsoft.com-nuget/sessions/784a9428-a98e-4087-9d10-030b796006a3 Co-authored-by: nkolev92 <2878341+nkolev92@users.noreply.github.com>
|
Learn Build status updates of commit 5f5cc25: ✅ Validation status: passed
For more details, please refer to the build report. |
nkolev92
left a comment
There was a problem hiding this comment.
@copilot Can you document NU1018 as well. NuGet/NuGet.Client@fbcb617 has details
… resolver Agent-Logs-Url: https://github.com/NuGet/docs.microsoft.com-nuget/sessions/8fa7f41c-6962-4b66-856f-5f5aaf431121 Co-authored-by: nkolev92 <2878341+nkolev92@users.noreply.github.com>
Added NU1018 documentation in commit ca7e199. Created |
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 ca7e199: ✅ 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 99d3a68: ✅ 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 |
Documents the new NU1018 and NU1019 errors introduced in NuGet.Client#7244 and NuGet.Client@fbcb617.
docs/reference/errors-and-warnings/NU1018.md: Documents the NU1018 error for duplicate target frameworks that require the default dependency resolver (.NET SDK 10.0.300+), including scenario description, framework aliasing context, and solution guidance.docs/reference/errors-and-warnings/NU1019.md: Documents the NU1019 error for disallowed characters inTargetFrameworkproperty values that would break the assets file format, including SDK analysis level gating (path separators error at 10.0.300+, non-ASCII warning→error at 11.0.100+) and solution guidance.docs/reference/Errors-and-Warnings.md: Added NU1018 and NU1019 to the "Invalid input errors" row.docs/TOC.md: Added NU1018, NU1019, and other missing error/warning pages (NU1013, NU1016, NU1017, NU5038, NU5039, NU5040, NU5041, NU5051, NU5133) to the table of contents.