Fix clang tidy diff (#141)
* Ignore header files for clang-tidy-diff * Rename code analysis job
This commit is contained in:
parent
59105f68bd
commit
50b6afd73d
4
.github/workflows/diff.yaml
vendored
4
.github/workflows/diff.yaml
vendored
@ -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
|
||||
|
Loading…
Reference in New Issue
Block a user