Skip to content

Update NU1703 doc to reflect new monoandroid warning#3542

Open
sbomer wants to merge 5 commits intoNuGet:mainfrom
sbomer:nu1704-doc
Open

Update NU1703 doc to reflect new monoandroid warning#3542
sbomer wants to merge 5 commits intoNuGet:mainfrom
sbomer:nu1704-doc

Conversation

@sbomer
Copy link
Copy Markdown

@sbomer sbomer commented Mar 25, 2026

Adds documentation for the warning added in NuGet/NuGet.Client#7229.

@sbomer sbomer requested review from a team as code owners March 25, 2026 22:48
Copilot AI review requested due to automatic review settings March 25, 2026 22:48
@learn-build-service-prod
Copy link
Copy Markdown

PoliCheck Scan Report

The 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 found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown

Learn Build status updates of commit df60919:

✅ Validation status: passed

File Status Preview URL Details
docs/reference/Errors-and-Warnings.md ✅Succeeded View
docs/reference/errors-and-warnings/NU1704.md ✅Succeeded View

For more details, please refer to the build report.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds documentation for the newly introduced NU1704 warning, and links it from the existing errors/warnings reference index so users can discover and understand the warning.

Changes:

  • Add a new reference page documenting warning NU1704, including cause, conditions, and suppression guidance.
  • Update the errors/warnings index table to include a link to NU1704.

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/NU1704.md New documentation page describing NU1704, its trigger conditions, and mitigation/suppression guidance.
docs/reference/Errors-and-Warnings.md Adds NU1704 to the “Package fallback warnings” row so it’s linked from the index.

Comment on lines +9 to +13
---

# NuGet Warning NU1704

> Package 'packageId' version uses the deprecated MonoAndroid framework instead of 'net6.0-android' or later. Consider upgrading to a newer version of this package or contacting the package author.
Copy link

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

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

The quoted warning text reads awkwardly/incorrectly: "Package 'packageId' version uses..." (it looks like the version value is missing). Update the message to either include the version placeholder/value or rephrase to remove "version" so the sentence is grammatically correct and matches the actual NU1704 warning output.

Suggested change
---
# NuGet Warning NU1704
> Package 'packageId' version uses the deprecated MonoAndroid framework instead of 'net6.0-android' or later. Consider upgrading to a newer version of this package or contacting the package author.
ai-usage: ai-generated
---
# NuGet Warning NU1704
> Package 'packageId' version 'version' uses the deprecated MonoAndroid framework instead of 'net6.0-android' or later. Consider upgrading to a newer version of this package or contacting the package author.

Copilot uses AI. Check for mistakes.
@Frulfump
Copy link
Copy Markdown

This part from the implementation PR seems to not be covered

The detection inspects compile-time and runtime assembly paths in the lock file target library for 'monoandroid' folder segments, following the pattern of the previously removed MaccatalystFallback helper.

Does that mean it will only show up if you use lockfiles?

@nkolev92
Copy link
Copy Markdown
Member

Due to historical reasons the type "LockFile" is the one that represents the assets file. So it'd be a massive breaking change to change that now, so often times people may refer to lock files, but they really mean the assets file.
The list of people that consistently use the distinct language is probably limited to the people that work on restore every day :D

@Frulfump
Copy link
Copy Markdown

Ah I see, thanks for the explanation!

Per PR feedback on NuGet/NuGet.Client#7229, reuse the NU1703 warning
code for the MonoAndroid deprecation scenario. The previous NU1703
for Xamarin.iOS never shipped, so the code is safe to repurpose.

Remove NU1704 doc and references from TOC and index.
@learn-build-service-prod
Copy link
Copy Markdown

PoliCheck Scan Report

The 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 found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown

Learn Build status updates of commit ea200b3:

💡 Validation status: suggestions

File Status Preview URL Details
docs/reference/errors-and-warnings/NU1703.md 💡Suggestion View Details
docs/reference/Errors-and-Warnings.md ✅Succeeded View

docs/reference/errors-and-warnings/NU1703.md

  • Line 5, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'kmarchan' is not a valid Microsoft alias.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@sbomer sbomer changed the title Add doc for NU1704 Update NU1703 doc to reflect new monoandroid warning Mar 30, 2026
@learn-build-service-prod
Copy link
Copy Markdown

PoliCheck Scan Report

The 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 found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown

Learn Build status updates of commit a8660b2:

✅ Validation status: passed

File Status Preview URL Details
docs/reference/Errors-and-Warnings.md ✅Succeeded View
docs/reference/errors-and-warnings/NU1703.md ✅Succeeded View

For more details, please refer to the build report.

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.

4 participants