Skip to content

Update clang-tidy.md - #5033

Merged
Jak Koke (Jak-MS) merged 4 commits into
MicrosoftDocs:mainfrom
Omotola:clang-tidy-update
May 22, 2024
Merged

Jak Koke (Jak-MS) merged 4 commits into
MicrosoftDocs:mainfrom
Omotola:clang-tidy-update

Conversation

@Omotola

@Omotola Omotola commented May 8, 2024

Copy link
Copy Markdown
Contributor

Added information for a new project property that allows selection of a custom clang-tidy exe

Added information for new tool directory property
@prmerger-automator

Copy link
Copy Markdown
Contributor

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

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 1ad0ecb:

✅ Validation status: passed

File Status Preview URL Details
docs/code-quality/clang-tidy.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Court72

Copy link
Copy Markdown
Contributor

Omotola,

When this PR is ready for review, remove Draft mode by selecting Ready for review.

image

@Court72

Copy link
Copy Markdown
Contributor

Tyler Whitney (@TylerMSFT)

Can you review the proposed changes?

When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

#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 May 9, 2024

@TylerMSFT Tyler Whitney (TylerMSFT) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

A couple minor suggestions.

Comment thread docs/code-quality/clang-tidy.md Outdated
Comment thread docs/code-quality/clang-tidy.md Outdated
minor corrections
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 930414f:

✅ Validation status: passed

File Status Preview URL Details
docs/code-quality/clang-tidy.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

grammar correction
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 2523ec1:

✅ Validation status: passed

File Status Preview URL Details
docs/code-quality/clang-tidy.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Omotola
Omotola marked this pull request as ready for review May 22, 2024 00:46

@TylerMSFT Tyler Whitney (TylerMSFT) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you for updating this topic

Comment thread docs/code-quality/clang-tidy.md Outdated
@Jak-MS
Jak Koke (Jak-MS) merged commit c2c9412 into MicrosoftDocs:main May 22, 2024
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 6cbac18:

✅ Validation status: passed

File Status Preview URL Details
docs/code-quality/clang-tidy.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

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.

4 participants