From 3b087c7119772cf47d349dd89ec44885f6866d04 Mon Sep 17 00:00:00 2001 From: Colin Davidson Date: Mon, 16 Oct 2023 10:36:21 +0100 Subject: [PATCH] Updrade clang-tidy to 17 --- .github/workflows/run_pr_tests.yml | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/.github/workflows/run_pr_tests.yml b/.github/workflows/run_pr_tests.yml index 04c986bed..9c60e2e4b 100644 --- a/.github/workflows/run_pr_tests.yml +++ b/.github/workflows/run_pr_tests.yml @@ -109,10 +109,6 @@ jobs: llvm_version: 16 llvm_build_type: RelAssert - # installs clang-tidy - - name: setup-clang-tidy - run: - sudo apt-get install -y clang-tidy-12 # These need to match the configurations of build_pr_cache to use the cache effectively - name: build initial config files @@ -160,7 +156,7 @@ jobs: HEAD..origin/${{ github.base_ref }} | \ grep -P '\.(c|cc|cxx|cpp|h|hh|hpp|hxx)$'` | \ tee /dev/stderr | \ - parallel --verbose -- clang-tidy-12 --quiet -p "$PWD/build/" "{}" + parallel --verbose -- clang-tidy-17 --quiet -p "$PWD/build/" "{}" # ^ When updating the clang-tidy version, the version used by the cmake # target should match updated c.f. the `tidy` target @@ -179,7 +175,7 @@ jobs: - name: run clang-format run: | - # we've installed clang-format-16 in the docker via pip, which just installs it as clang-format, + # we've installed clang-format-17 in the docker via pip, which just installs it as clang-format, # so just use clang-format-diff and -b clang-format directly git fetch origin ${{ github.base_ref }} git diff --no-color origin/${{ github.base_ref }} | \