Structure warning references in range [C26800, C26830]#5805
Conversation
|
@Rageking8 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit a8a8761: ✅ Validation status: passed
For more details, please refer to the build report. |
PRMerger Results
|
There was a problem hiding this comment.
Pull Request Overview
This PR adds quotes around warning titles in YAML front matter for warning references in the C26800-C26830 range. This change standardizes the format of title fields across documentation files.
- Adds quotes around warning titles in YAML front matter
- Reorders metadata fields to follow a consistent structure
- Reorganizes content placement for better document flow
Reviewed Changes
Copilot reviewed 21 out of 21 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| c26800.md | Adds quotes around warning title |
| c26810.md | Adds quotes around warning title |
| c26811.md | Adds quotes around warning title |
| c26812.md | Reorders title and moves to line 2 with quotes |
| c26813.md | Adds quotes around warning title |
| c26814.md | Reorders title to line 2 and adds quotes |
| c26815.md | Reorders ms.date field and adds quotes to title |
| c26816.md | Reorders ms.date field and adds quotes to title |
| c26817.md | Adds quotes to title and inserts Remarks heading |
| c26818.md | Adds quotes around warning title |
| c26819.md | Adds quotes around warning title |
| c26820.md | Moves Remarks section and adds quotes to title |
| c26822.md | Adds quotes around warning title |
| c26823.md | Adds quotes around warning title |
| c26824.md | Adds quotes around warning title |
| c26825.md | Adds quotes around warning title |
| c26826.md | Moves content and adds quotes to title |
| c26827.md | Adds quotes around warning title |
| c26828.md | Adds quotes around warning title |
| c26829.md | Adds quotes around warning title |
| c26830.md | Adds quotes around warning title |
|
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
There was a problem hiding this comment.
The rules for metadata are tricky. We only use quotation marks when the value includes a colon (:) or a hash (#) or something else that needs to be escaped. (https://learn.microsoft.com/en-us/help/platform/metadata-required-articles)
I'm going to close this PR but I will add the #Remarks section to the topics you touched.
I'm also looking into whether it matters. I'll reopen this if not. I suspect it doesn't as we have both examples in the repo.
|
Reopening. It doesn't "hurt" to put the title in quotes. And I want the Remarks changes. |
|
Learn Build status updates of commit a8a8761: ✅ Validation status: passed
For more details, please refer to the build report. |
PRMerger Results
|
|
#sign-off |
This is batch 120 that structures error/warning references. See #5465 for more information.