Skip to content

Commit 2d6a4db

Browse files
Takuto IkutaChromium LUCI CQ
authored andcommitted
.clang-tidy: ignore system headers from misc-include-cleaner
It seems that we don't want to add/remove system headers in some cases. e.g. https://crrev.com/c/5524195/5/crypto/nss_key_util.cc Bug: 336474469 Change-Id: I56b4239832a5246cb6edb39bf6397ceb03e0a003 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5528895 Reviewed-by: Peter Kasting <pkasting@chromium.org> Reviewed-by: Nico Weber <thakis@chromium.org> Commit-Queue: Takuto Ikuta <tikuta@chromium.org> Auto-Submit: Takuto Ikuta <tikuta@chromium.org> Cr-Commit-Position: refs/heads/main@{#1299890}
1 parent 12e31f1 commit 2d6a4db

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

.clang-tidy

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,9 @@
6666
value: base/containers/adapters.h
6767
# Exclude some third_party headers from modification as file paths are not
6868
# starting from repository root in replacement suggestion.
69+
# 'build/linux/debian' excludes system headers as they don't have
70+
# appropriate IWYU pragmas.
6971
# https://clang.llvm.org/extra/clang-tidy/checks/misc/include-cleaner.html
7072
- key: misc-include-cleaner.IgnoreHeaders
71-
value: (gmock/gmock|gtest/gtest|third_party).*
73+
value: (gmock/gmock|gtest/gtest|third_party|build/linux/debian).*
7274
...

0 commit comments

Comments
 (0)