[PATCH] D91602: [clang-tidy] Make clang-format and include-order-check coherent

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 17 03:39:08 PST 2020


njames93 added a comment.

Thanks for fixing this, been bugging me for a while.



================
Comment at: clang-tools-extra/test/clang-tidy/checkers/llvm-include-order.cpp:1
 // RUN: %check_clang_tidy %s llvm-include-order %t -- -- -isystem %S/Inputs/Headers
 
----------------
Would it be wise to specify a format style here to ensure they align. Better yet 2 run lines, one that runs clang-tidy with no format, the other specifies a format.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D91602/new/

https://reviews.llvm.org/D91602



More information about the cfe-commits mailing list