-
Notifications
You must be signed in to change notification settings - Fork 967
Add some warning entries #5018
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
Add some warning entries #5018
Conversation
@Rageking8 : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Learn Build status updates of commit a6d81cc: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Learn Build status updates of commit 8c96918: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
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.
This is good work. Thank you.
#sign-off |
One way to fix that issue is to break that article into 2 or more articles. This way no redirect is needed since the article retains it filename, while the excess warnings can be placed in a new article (keeping to the existing 200 warnings/article chunking). Although there will be no link breakage (or redirects), there will still be that "semantic" difference where someone links to that article but are referring to the C5000+ range, which I would argue is rare and not that impactful. |
#sign-off |
Summary:
f1_keywords
.f1_keywords
.f1_keywords
andhelpviewer_keywords
.Note 1) Normalization of table formatting can be done in a separate PR (maybe to the style of
compiler-warnings-c4800-through-c4999.md
, i.e. with spaces around the table cells).Note 2) Not sure why article "Compiler warnings C4800 through C5999" has name
compiler-warnings-c4800-through-c4999.md
(this affects the article link as well).