Skip to content

Commit

Permalink
ci: fail clang-tidy job on threshold
Browse files Browse the repository at this point in the history
  • Loading branch information
battlmonstr committed Nov 13, 2024
1 parent 3984398 commit f691509
Show file tree
Hide file tree
Showing 5 changed files with 38 additions and 586 deletions.
12 changes: 8 additions & 4 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ commands:
command: |
if [[ <<parameters.ethereum_tests>> == false ]]
then
git config submodule.ethereum-tests.update none
git config submodule.ethereum-tests.update none
fi
git submodule sync
Expand Down Expand Up @@ -71,12 +71,13 @@ commands:
cmake ../project -DCMAKE_BUILD_TYPE=<<parameters.build_type>> $CONAN_CMAKE_ARGS $TOOLCHAIN_CMAKE_ARGS $BUILD_CMAKE_ARGS
- run:
name: "Build"
command: |
working_directory: ~/build
command: |
if [[ "<<parameters.target>>" == "all" ]]
then
cmake --build ~/build -j16
cmake --build . -j16 |& tee build.log
else
cmake --build ~/build -j16 --target <<parameters.target>>
cmake --build . -j16 --target <<parameters.target>> |& tee build.log
fi
install_conan:
Expand Down Expand Up @@ -344,6 +345,9 @@ jobs:
build_type: Debug
compiler_id: clang
compiler_version: <<pipeline.parameters.clang_version_latest>>
- run:
name: "Report"
command: tools/lint/clang_tidy_report.sh "$HOME/build/build.log"

linux-clang-tidy-diff:
environment:
Expand Down
2 changes: 1 addition & 1 deletion .clang-tidy
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
FormatStyle: file
HeaderFilterRegex: 'silkworm/(capi|core|db|execution|infra|node|rpc|sentry|sync|wasm)/.+\.hpp$'
WarningsAsErrors: '*'
WarningsAsErrors: 'bugprone-use-after-move'
Checks: >
abseil-*,
boost-*,
Expand Down
23 changes: 0 additions & 23 deletions third_party/clang-tidy-html/LICENSE

This file was deleted.

Loading

0 comments on commit f691509

Please sign in to comment.