erenon / bazel_clang_tidy

Run clang-tidy on Bazel C++ targets directly, efficiently, with caching enabled
MIT License
109 stars 58 forks source link

Add external includes to clang-tidy invocation #68

Closed michael-ahn closed 1 month ago

michael-ahn commented 2 months ago
eneev-a commented 1 month ago

This would be good to get in:

https://bazel.build/rules/lib/builtins/CompilationContext#external_includes says that these external_includes usually included with angle brackets:

depset CompilationContext.external_includes Returns the set of search paths (as strings) for external header files referenced by angle bracket. Usually passed with -isystem.

dotnwat commented 1 month ago

+1

erenon commented 1 month ago

Thanks! Merged.