-
Notifications
You must be signed in to change notification settings - Fork 967
Update clang-tidy.md #4948
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
Update clang-tidy.md #4948
Conversation
Adding support for CMake Presets
@sinemakinci1 : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Learn Build status updates of commit 0c23813: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PRMerger Results
|
#label:"aq-pr-triaged" |
addressing some feedback
Learn Build status updates of commit ea7939d: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PRMerger Results
|
@TylerMSFT can you take a look at the updates now? |
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.
Thanks, Sinem! Left a couple comments.
@sinemakinci1 : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
@TylerMSFT updated, thanks for your additional review! |
Learn Build status updates of commit a52b023: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PRMerger Results
|
Learn Build status updates of commit 01ec199: ✅ Validation status: passed
For more details, please refer to the build 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.
Sorry - I missed a tagging issue for the code blocks.
Learn Build status updates of commit 6de2ecf: ✅ Validation status: passed
For more details, please refer to the build 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.
Thanks, Sinem!
Learn Build status updates of commit d90f8ad: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PRMerger Results
|
Learn Build status updates of commit 474bf33: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PRMerger Results
|
Adding support for CMake Presets