From 50b6afd73de31c95e81fd4eda9cd9f9935d505fb Mon Sep 17 00:00:00 2001 From: antonio2368 <antonio2368@users.noreply.github.com> Date: Wed, 21 Apr 2021 16:26:13 +0200 Subject: [PATCH] Fix clang tidy diff (#141) * Ignore header files for clang-tidy-diff * Rename code analysis job --- .github/workflows/diff.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/diff.yaml b/.github/workflows/diff.yaml index 502052f94..9447ab957 100644 --- a/.github/workflows/diff.yaml +++ b/.github/workflows/diff.yaml @@ -65,7 +65,7 @@ jobs: name: "Community DEB package" path: build/output/memgraph*.deb - coverage_build: + code_analysis: name: "Code analysis" runs-on: [self-hosted, General, Linux, X64, Debian10] env: @@ -124,7 +124,7 @@ jobs: source /opt/toolchain-v2/activate # Restrict clang-tidy results only to the modified parts - git diff -U0 master... -- src | ./tools/github/clang-tidy/clang-tidy-diff.py -p 1 -j $THREADS -path build | tee ./build/clang_tidy_output.txt + git diff -U0 master... -- src ':!*.hpp' | ./tools/github/clang-tidy/clang-tidy-diff.py -p 1 -j $THREADS -path build | tee ./build/clang_tidy_output.txt # Fail if any warning is reported ! cat ./build/clang_tidy_output.txt | ./tools/github/clang-tidy/grep_error_lines.sh > /dev/null