-
Notifications
You must be signed in to change notification settings - Fork 11.9k
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
llvm-include-order check inconsistent with clang-format #53525
Labels
Comments
@llvm/issue-subscribers-clang-tidy |
llvmbot
pushed a commit
to llvmbot/llvm-project
that referenced
this issue
Feb 4, 2022
Clang-format LLVM style has a custom include category for gtest/ and gmock/ headers between regular includes and angled includes. Do the same here. Fixes llvm#53525. Differential Revision: https://reviews.llvm.org/D118913 (cherry picked from commit 0447ec2)
llvmbot
pushed a commit
to llvmbot/llvm-project
that referenced
this issue
Feb 5, 2022
Clang-format LLVM style has a custom include category for gtest/ and gmock/ headers between regular includes and angled includes. Do the same here. Fixes llvm#53525. Differential Revision: https://reviews.llvm.org/D118913 (cherry picked from commit 0447ec2)
tstellar
pushed a commit
that referenced
this issue
Feb 7, 2022
Clang-format LLVM style has a custom include category for gtest/ and gmock/ headers between regular includes and angled includes. Do the same here. Fixes #53525. Differential Revision: https://reviews.llvm.org/D118913 (cherry picked from commit 0447ec2)
mem-frob
pushed a commit
to draperlaboratory/hope-llvm-project
that referenced
this issue
Oct 7, 2022
Clang-format LLVM style has a custom include category for gtest/ and gmock/ headers between regular includes and angled includes. Do the same here. Fixes llvm/llvm-project#53525. Differential Revision: https://reviews.llvm.org/D118913
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
clang-format wants:
(gtest is a group above system headers)
clang-tidy wants:
(gtest is grouped within system headers)
I think this went wrong in 5a9f386
The text was updated successfully, but these errors were encountered: