Skip to content

Commit 026a763

Browse files
simon-d-bmwrolandreichweinbmw
authored andcommitted
Adjust clang-tidy workflow scope
Exclude test and mock paths from the clang-tidy workflow so the CT-P1 enablement can be reviewed independently from production-code suppression and remediation work. The workflow file was validated for YAML syntax locally.
1 parent 4f3a4bf commit 026a763

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

.github/workflows/clang-tidy.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,10 +90,10 @@ jobs:
9090
9191
if [[ "${CLANG_TIDY_MODE}" == "full" ]]; then
9292
DOCKER_UID=$(id -u) DOCKER_GID=$(id -g) DOCKER_HISTORY=/dev/null docker compose run --rm development \
93-
python3 .ci/clang-tidy.py --build_directory=build/tests/${{ matrix.platform }}/${{ matrix.config }} --output_file=ct-findings.yaml --exclude="3rdparty" --ignore-checks="clang-diagnostic-error"
93+
python3 .ci/clang-tidy.py --build_directory=build/tests/${{ matrix.platform }}/${{ matrix.config }} --output_file=ct-findings.yaml --exclude="3rdparty|/test/|/mock/" --ignore-checks="clang-diagnostic-error"
9494
else
9595
DOCKER_UID=$(id -u) DOCKER_GID=$(id -g) DOCKER_HISTORY=/dev/null docker compose run --rm development \
96-
python3 .ci/clang-tidy.py --build_directory=build/tests/${{ matrix.platform }}/${{ matrix.config }} --output_file=ct-findings.yaml --exclude="3rdparty" --ignore-checks="clang-diagnostic-error" --file-list=.clang-tidy-changed-files.txt
96+
python3 .ci/clang-tidy.py --build_directory=build/tests/${{ matrix.platform }}/${{ matrix.config }} --output_file=ct-findings.yaml --exclude="3rdparty|/test/|/mock/" --ignore-checks="clang-diagnostic-error" --file-list=.clang-tidy-changed-files.txt
9797
fi
9898
9999
- name: Skip clang-tidy because no C/C++ files changed

0 commit comments

Comments
 (0)