Skip to content

Commit aad7ac8

Browse files
committed
update iwyu warning limits. use cmake args to trigger clang-tidy. disable opentracing with clang-tidy builds until the cmake linking for that target is fixed.
1 parent e807e3b commit aad7ac8

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

.github/workflows/clang-tidy.yaml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -13,13 +13,13 @@ jobs:
1313
clang-tidy:
1414
runs-on: ubuntu-24.04
1515
strategy:
16-
fail-fast: false
16+
fail-fast: true
1717
matrix:
1818
include:
1919
- cmake_options: all-options-abiv1-preview
20-
warning_limit: 0
20+
warning_limit: 236
2121
- cmake_options: all-options-abiv2-preview
22-
warning_limit: 216
22+
warning_limit: 236
2323
steps:
2424
- name: Harden the runner (Audit all outbound calls)
2525
uses: step-security/harden-runner@0634a2670c59f64b4a01f0f96f84700a4088b9f0 # v2.12.0
@@ -60,7 +60,6 @@ jobs:
6060
echo "Using clang-tidy version: $(clang-tidy --version)"
6161
echo "clang-tidy installed at: $(which clang-tidy)"
6262
63-
6463
- name: Prepare CMake
6564
env:
6665
CC: clang
@@ -71,12 +70,13 @@ jobs:
7170
-C ./test_common/cmake/${{ matrix.cmake_options }}.cmake \
7271
-DCMAKE_CXX_STANDARD=14 \
7372
-DWITH_STL=CXX14 \
74-
-DCMAKE_EXPORT_COMPILE_COMMANDS=ON
73+
-DWITH_OPENTRACING=OFF \
74+
-DCMAKE_EXPORT_COMPILE_COMMANDS=ON \
75+
-DCMAKE_CXX_CLANG_TIDY="clang-tidy;--quiet;-p;build-${{ matrix.cmake_options }}"
7576
7677
- name: Run clang-tidy
7778
run: |
78-
cmake --build build-${{ matrix.cmake_options }}
79-
jq -r .[].file build-${{ matrix.cmake_options }}/compile_commands.json | grep -vE '/(generated|third_party)/' | xargs -P $(nproc) -n 1 clang-tidy --quiet -p build-${{ matrix.cmake_options }} 2>&1 | tee -a clang-tidy-${{ matrix.cmake_options }}.log
79+
cmake --build build-${{ matrix.cmake_options }} -- -j$(nproc) 2>&1 | tee clang-tidy-${{ matrix.cmake_options }}.log
8080
8181
- uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
8282
with:

.github/workflows/iwyu.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,13 +14,13 @@ jobs:
1414
iwyu:
1515
runs-on: ubuntu-latest
1616
strategy:
17-
fail-fast: false
17+
fail-fast: true
1818
matrix:
1919
include:
2020
- cmake_options: all-options-abiv1
2121
warning_limit: 3
2222
- cmake_options: all-options-abiv1-preview
23-
warning_limit: 48
23+
warning_limit: 28
2424
- cmake_options: all-options-abiv2-preview
2525
warning_limit: 48
2626

0 commit comments

Comments
 (0)