Skip to content

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

Merged
merged 3 commits into from
Sep 15, 2022

Conversation

bradlitterell
Copy link
Contributor

@corob-msft
proposal. I don't warrant it shouldn't be worded in a better way.

@corob-msft 
proposal.  I don't warrant it shouldn't be worded in a better way.
@prmerger-automator
Copy link
Contributor

@bradlitterell : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@opbld30
Copy link

opbld30 commented Sep 2, 2022

Docs Build status updates of commit 091596c:

✅ Validation status: passed

File Status Preview URL Details
docs/error-messages/compiler-warnings/compiler-warning-level-4-c4464.md ✅Succeeded

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-automator
Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 20% file change.

@prmerger-automator
Copy link
Contributor

@bradlitterell : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@opbld32
Copy link

opbld32 commented Sep 2, 2022

Docs Build status updates of commit c658907:

✅ Validation status: passed

File Status Preview URL Details
docs/error-messages/compiler-warnings/compiler-warning-level-4-c4464.md ✅Succeeded

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-automator
Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 20% file change.

@Jak-MS
Copy link
Contributor

Jak-MS commented Sep 6, 2022

@corob-msft

  • Can you review this PR?

IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged Tracking label for the PR review team label Sep 6, 2022
@bradlitterell
Copy link
Contributor Author

@Jak-MS Are you saying I need to add the #sign-off in a comment, or @corob-msft does? Thanks,

@prmerger-automator
Copy link
Contributor

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @corob-msft

@Jak-MS
Copy link
Contributor

Jak-MS commented Sep 6, 2022

@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.
@prmerger-automator
Copy link
Contributor

@corob-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@opbld33
Copy link

opbld33 commented Sep 15, 2022

Docs Build status updates of commit 614978c:

✅ Validation status: passed

File Status Preview URL Details
docs/error-messages/compiler-warnings/compiler-warning-level-4-c4464.md ✅Succeeded

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-automator
Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 20% file change.

Copy link
Contributor

@colin-home colin-home left a 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.

@colin-home colin-home merged commit a3b5a24 into MicrosoftDocs:main Sep 15, 2022
@bradlitterell
Copy link
Contributor Author

thanks a bunch!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants