-
Notifications
You must be signed in to change notification settings - Fork 967
Update compiler-warning-level-4-c4464.md #4135
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
Conversation
@corob-msft proposal. I don't warrant it shouldn't be worded in a better way.
@bradlitterell : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Docs Build status updates of commit 091596c: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
PRMerger Results
|
@bradlitterell : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Docs Build status updates of commit c658907: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
PRMerger Results
|
@corob-msft
IMPORTANT: When this content is ready to merge, you must add #label:"aq-pr-triaged" |
@Jak-MS Are you saying I need to add the #sign-off in a comment, or @corob-msft does? Thanks, |
Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @corob-msft |
@bradlitterell - @corob-msft will need to sign off after he reviews the pr. Thanks. |
Updates for style and clarity. Also give it an Acrolinx pass.
@corob-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Docs Build status updates of commit 614978c: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
PRMerger Results
|
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.
@bradlitterell
Thanks for taking the initiative here, and for being patient while I got around to approving it. I gave the topic a more complete rewrite to bring it to current editorial standards and to clarify other issues you brought up in discussion as well.
thanks a bunch! |
@corob-msft
proposal. I don't warrant it shouldn't be worded in a better way.