Skip to content

Update compiler-warning-level-4-c4464.md - #4135

Merged
Colin Robertson (colin-home) merged 3 commits into
MicrosoftDocs:mainfrom
bradlitterell:patch-1
Sep 15, 2022
Merged

Colin Robertson (colin-home) merged 3 commits into
MicrosoftDocs:mainfrom
bradlitterell:patch-1

Conversation

@bradlitterell

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

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

@opbld30

opbld30 commented Sep 2, 2022

Copy link
Copy Markdown

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
Copy Markdown
Contributor

PRMerger Results

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

@prmerger-automator

Copy link
Copy Markdown
Contributor

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

@opbld32

opbld32 commented Sep 2, 2022

Copy link
Copy Markdown

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
Copy Markdown
Contributor

PRMerger Results

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

@Jak-MS

Copy link
Copy Markdown
Contributor

@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
Copy Markdown
Contributor Author

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

@prmerger-automator

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

Brad Litterell (@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
Copy Markdown
Contributor

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

@opbld33

opbld33 commented Sep 15, 2022

Copy link
Copy Markdown

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
Copy Markdown
Contributor

PRMerger Results

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Brad Litterell (@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 Robertson (colin-home) merged commit a3b5a24 into MicrosoftDocs:main Sep 15, 2022
@bradlitterell

Copy link
Copy Markdown
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