From f42c23218daed64026739c422be7db80fda4ef8c Mon Sep 17 00:00:00 2001 From: Ivan Grokhotkov Date: Thu, 14 Apr 2022 15:01:47 +0200 Subject: [PATCH] ci: clang-tidy fixes - upgrade clang-tidy-runner to 0.2.0. - use new '--include-paths' and '--exclude-paths' arguments of clang-tidy-runner to limit the issues reported. - don't install codereport and don't build an HTML report, fixing the Jinja2 version incompatibility issue. --- .github/workflows/clang-tidy.yml | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/.github/workflows/clang-tidy.yml b/.github/workflows/clang-tidy.yml index d20eda9..ab9253c 100644 --- a/.github/workflows/clang-tidy.yml +++ b/.github/workflows/clang-tidy.yml @@ -25,16 +25,15 @@ jobs: ${IDF_PATH}/tools/idf_tools.py --non-interactive install xtensa-clang . ${IDF_PATH}/export.sh which -a clang-tidy || true - pip install pyclang codereport + pip install pyclang~=0.2.0 curl -sSL https://raw.githubusercontent.com/espressif/llvm-project/xtensa_release_12.0.1/clang-tools-extra/clang-tidy/tool/run-clang-tidy.py -o run-clang-tidy.py chmod +x run-clang-tidy.py curl -sSL https://github.com/psastras/sarif-rs/releases/download/clang-tidy-sarif-latest/clang-tidy-sarif-x86_64-unknown-linux-gnu -o clang-tidy-sarif chmod +x clang-tidy-sarif export PATH=$PWD:$PATH - idf.py clang-check - idf.py clang-html-report + idf.py clang-check --include-paths $GITHUB_WORKSPACE --exclude-paths $PWD ./clang-tidy-sarif -o results.sarif.raw warnings.txt - python3 $GITHUB_WORKSPACE/.github/filter_sarif.py -o results.sarif --include-prefix ${GITHUB_WORKSPACE}/test_app/managed_components/espressif__ --include-prefix ${GITHUB_WORKSPACE}/ results.sarif.raw + python3 $GITHUB_WORKSPACE/.github/filter_sarif.py -o results.sarif --include-prefix ${GITHUB_WORKSPACE}/ results.sarif.raw cp results.sarif ../ cp results.sarif.raw ../ cp warnings.txt ../