-
Notifications
You must be signed in to change notification settings - Fork 972
Structure warning references in range [C26100, C26167] #5803
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
base: main
Are you sure you want to change the base?
Structure warning references in range [C26100, C26167] #5803
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 1e38532: ✅ Validation status: passed
For more details, please refer to the build report. |
PRMerger Results
|
Can you review the proposed changes? Editorial changes LGTM Important: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR restructures warning documentation files for C++ code analysis warnings in the range C26100-C26167 to follow a consistent format structure. The changes standardize the YAML front matter by moving the title field to the second line and removing obsolete ms.assetid fields, while also adding "## Remarks" section headers where they were missing.
Key changes:
- Restructured YAML front matter for consistency across warning documentation files
- Added missing "## Remarks" section headers to provide better content organization
- Removed obsolete ms.assetid metadata fields to clean up documentation structure
Reviewed Changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
c26100.md | Updated YAML front matter structure and removed obsolete ms.assetid |
c26101.md | Restructured front matter and added "## Remarks" section header |
c26105.md | Updated YAML structure and added "## Remarks" section |
c26110.md | Restructured front matter and added "## Remarks" section header |
c26111.md | Updated YAML structure and added "## Remarks" section |
c26112.md | Restructured front matter and added "## Remarks" section header |
c26115.md | Updated YAML structure and added "## Remarks" section |
c26117.md | Restructured front matter and added "## Remarks" section header |
c26130.md | Updated YAML structure and added "## Remarks" section |
c26135.md | Restructured front matter and added "## Remarks" section header |
c26138.md | Updated YAML front matter structure only |
c26140.md | Restructured front matter and removed obsolete ms.assetid |
c26160.md | Updated YAML structure and added "## Remarks" section |
c26165.md | Restructured front matter and added "## Remarks" section header |
c26166.md | Updated YAML structure and added "## Remarks" section |
c26167.md | Restructured front matter and added "## Remarks" section header |
C26116, C26132, and C26133 are skipped to prevent potential merge conflicts with #5591 and #5714. A separate PR will be issued once those are merged.
This is batch 119 that structures error/warning references. See #5465 for more information.