From 4f47631525380cb6bb4131153c0872380cfaba01 Mon Sep 17 00:00:00 2001 From: Sean Robinson Date: Tue, 29 Nov 2022 09:21:45 -0700 Subject: [PATCH 1/2] Update clang-tidy configuration to only include main header The previous regex would also analyze doctest.hpp. We don't need to check a vendored library. Signed-off-by: Sean Robinson --- .clang-tidy | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.clang-tidy b/.clang-tidy index d0b28f4..7d0c827 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -18,4 +18,4 @@ CheckOptions: - { key: readability-identifier-naming.StructIgnoredRegexp, value: "parse_number" } - { key: readability-identifier-naming.VariableCase, value: lower_case } -HeaderFilterRegex: '.*' +HeaderFilterRegex: 'argparse/.+\.hpp' From a832ac3496e1aeaf66b9f839a15c820c0d517d4e Mon Sep 17 00:00:00 2001 From: Sean Robinson Date: Tue, 29 Nov 2022 09:25:39 -0700 Subject: [PATCH 2/2] Analyze only argparse header in PR clang-tidy action Signed-off-by: Sean Robinson --- .github/workflows/tidy-analysis-stage-01.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/tidy-analysis-stage-01.yml b/.github/workflows/tidy-analysis-stage-01.yml index 5019152..66ca0a1 100644 --- a/.github/workflows/tidy-analysis-stage-01.yml +++ b/.github/workflows/tidy-analysis-stage-01.yml @@ -28,7 +28,7 @@ jobs: run: mkdir clang-tidy-result - name: Analyze - run: git diff -U0 HEAD^ | clang-tidy-diff-12.py -p1 -path build -extra-arg=-Iinclude -extra-arg=-std=c++17 -export-fixes clang-tidy-result/fixes.yml + run: git diff -U0 HEAD^ | clang-tidy-diff-12.py -p1 -regex ".+hpp" -extra-arg=-Iinclude -extra-arg=-std=c++17 -export-fixes clang-tidy-result/fixes.yml - name: Save PR metadata run: |