From 414d54a28b749af6abcaec6b85c99cc69de6b7dd Mon Sep 17 00:00:00 2001 From: Heejin Ahn Date: Wed, 29 Sep 2021 10:54:57 -0700 Subject: Exclude test/ from clang-tidy (#4193) We don't keep files within test/ clang-tidy compliant, so including this directory can generate CI errors when test files change. --- scripts/clang-tidy-diff.sh | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'scripts/clang-tidy-diff.sh') diff --git a/scripts/clang-tidy-diff.sh b/scripts/clang-tidy-diff.sh index 0e0882703..0d0c367ba 100755 --- a/scripts/clang-tidy-diff.sh +++ b/scripts/clang-tidy-diff.sh @@ -24,15 +24,16 @@ function realpath() { CLANG_DIR=$(dirname $(dirname $(realpath $CLANG_TIDY))) CLANG_TIDY_DIFF=$CLANG_DIR/share/clang/clang-tidy-diff.py +ARG="-quiet -p1 -regex=src/.*" if [ ! -e "$CLANG_TIDY_DIFF" ]; then echo "Failed to find clang-tidy-diff.py ($CLANG_TIDY_DIFF)" exit 1 fi -TIDY_MSG=$(git diff -U0 $BRANCH... | $CLANG_TIDY_DIFF -quiet -p1 2> /dev/null) +TIDY_MSG=$(git diff -U0 $BRANCH... | $CLANG_TIDY_DIFF $ARG 2> /dev/null) if [ -n "$TIDY_MSG" -a "$TIDY_MSG" != "No relevant changes found." ]; then echo "Please fix clang-tidy errors before committing" echo # Run clang-tidy once again to show the error - git diff -U0 $BRANCH... | $CLANG_TIDY_DIFF -quiet -p1 2> /dev/null + git diff -U0 $BRANCH... | $CLANG_TIDY_DIFF $ARG 2> /dev/null exit 1 fi -- cgit v1.2.3