Skip to content

Commit f4b3868

Browse files
committed
Run clang-tidy action only for clang builds
No need to comment twice
1 parent efaae33 commit f4b3868

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

.github/workflows/tests.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ jobs:
2929
run: |
3030
cppcheck --std=c++11 --enable=performance,portability,information --quiet --error-exitcode=1 src/
3131
- name: Run clang-tidy
32+
if: matrix.compiler.cc == 'clang-10'
3233
run: |
3334
run-clang-tidy-8 \
3435
-p=test/build \
@@ -38,12 +39,13 @@ jobs:
3839
-export-fixes clang-tidy-fixes.yaml
3940
- name: Run clang-tidy-pr-comments action
4041
uses: platisd/clang-tidy-pr-comments@master
42+
if: matrix.compiler.cc == 'clang-10'
4143
with:
4244
github_token: ${{ secrets.GITHUB_TOKEN }}
4345
clang_tidy_fixes: clang-tidy-fixes.yaml
4446
request_changes: true
4547
- name: Get coverage report (only for GCC)
46-
if: ${{ matrix.compiler.cc }} == 'gcc-10'
48+
if: matrix.compiler.cc == 'gcc-10'
4749
run: |
4850
bash run_coverage.sh
4951
bash <(curl -s https://codecov.io/bash) -s coverage-reports || echo "Codecov did not collect coverage reports"

0 commit comments

Comments
 (0)