-
Notifications
You must be signed in to change notification settings - Fork 239
Address PR feedback: refine XAML source generation documentation #3079
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
Address PR feedback: refine XAML source generation documentation #3079
Conversation
|
Learn Build status updates of commit a171d51: ✅ Validation status: passedFor more details, please refer to the build report. |
- Changed "preview" to "opt-in" in IMPORTANT notes - Reordered content to promote global setting first - Added fine-grained control section with clear context - Fixed markdown formatting (missing > characters in runtime-load.md) Co-authored-by: jfversluis <939291+jfversluis@users.noreply.github.com>
|
Learn Build status updates of commit 948a1bf: ✅ Validation status: passed
For more details, please refer to the build report. |
|
Learn Build status updates of commit 948a1bf: ✅ 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 |
8dfa4e2
into
pr14-xaml-sourcegen-global-xmlns-net10-from-main
* docs(.NET10): XAML preview features — source generator and global xmlns; add admonitions and examples; update ms.date * revised source gen bits * Update get-started.md * Update get-started.md with XAML source generation info Added instructions for enabling XAML source generation and improved namespace usage in .NET MAUI. * Revise XAML source generation instructions for .NET MAUI Updated instructions for enabling XAML source generation in .NET MAUI for .NET 10, including new examples and clarifications. * Address PR feedback: refine XAML source generation documentation (#3079) * Initial plan * Address PR feedback: update XAML source generation docs - Changed "preview" to "opt-in" in IMPORTANT notes - Reordered content to promote global setting first - Added fine-grained control section with clear context - Fixed markdown formatting (missing > characters in runtime-load.md) Co-authored-by: jfversluis <939291+jfversluis@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: jfversluis <939291+jfversluis@users.noreply.github.com> * Update docs/xaml/runtime-load.md * Update docs/xaml/fundamentals/get-started.md Co-authored-by: Stephane Delcroix <stephane@delcroix.org> * Apply suggestions from code review --------- Co-authored-by: Gerald Versluis <gerald.versluis@microsoft.com> Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com> Co-authored-by: jfversluis <939291+jfversluis@users.noreply.github.com> Co-authored-by: Stephane Delcroix <stephane@delcroix.org>
Updated XAML source generation documentation based on review feedback to correct terminology and improve content organization.
Changes
MauiXamlInflator) as primary approach, repositioned per-file configuration as secondary option for fine-grained controlRecommended approach now reads:
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.
Internal previews