search

erenon / bazel_clang_tidy

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

Allow treating /_virtual_includes/ and bazel-out/ as -isystem #53

Open lalten opened 1 year ago

lalten commented 1 year ago

Like https://github.com/erenon/bazel_clang_tidy/pull/32, but as opt-in behavior. Additionally allow ignoring generated includes in bazel-out/

The opt-in part is handled via bool_flags, which needs Skylib.

lalten commented 9 months ago

ping on this @erenon, wdyt?

erenon commented 9 months ago

Hi, since a recent change, HeaderFilterRegex is no longer recommended: https://github.com/erenon/bazel_clang_tidy/commit/f43f9d61c201b314c62a3ebcf2d4a37f1a3b06f7

Instead, lib headers are added automatically. Does this affect this change?

re skylib: I'd like to keep this project dependency free as possible.