From f735ec6ff6fbb5ea66444a896d133c822d75eb08 Mon Sep 17 00:00:00 2001 From: Chun Cai Date: Mon, 17 Jan 2022 15:47:41 +0800 Subject: [PATCH] Test: fix code static analysis not working. --- .github/workflows/static.yaml | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/.github/workflows/static.yaml b/.github/workflows/static.yaml index d535c2356e..154043f5d0 100644 --- a/.github/workflows/static.yaml +++ b/.github/workflows/static.yaml @@ -5,7 +5,7 @@ on: jobs: clang-tidy: - runs-on: ubuntu-20.04 + runs-on: ubuntu-latest container: ghcr.io/deepmodeling/abacus-development-kit:gnu steps: - name: Checkout Pull Request @@ -14,22 +14,17 @@ jobs: ref: ${{ github.event.pull_request.head.sha }} - name: Setup clang-tidy run: | - apt-get install -y --no-install-recommends gnupg ca-certificates - wget https://apt.llvm.org/llvm-snapshot.gpg.key --no-check-certificate && apt-key add llvm-snapshot.gpg.key - echo "deb http://apt.llvm.org/buster/ llvm-toolchain-buster-12 main" >> /etc/apt/sources.list - echo "deb-src http://apt.llvm.org/buster/ llvm-toolchain-buster-12 main" >> /etc/apt/sources.list - apt-get update - apt-get install -y --no-install-recommends clang-tidy-12 + apt-get update && apt-get install -y --no-install-recommends clang-format clang-tidy - name: Generate Build Commands run: | - cmake -B build -DCMAKE_EXPORT_COMPILE_COMMANDS=1 -DCMAKE_CXX_COMPILER=clang++-12 + cmake -B build -DCMAKE_EXPORT_COMPILE_COMMANDS=1 - name: Checkout Changed Files id: files uses: deepmodeling/get-changed-files@master - name: Run clang-tidy run: | printf "%s\n" ${{ steps.files.outputs.added_modified }} | grep -E "^.*\.(cpp|h)$" > cppfiles.txt - clang-tidy-12 -p=build --export-fixes=diagnostics.yaml `cat cppfiles.txt` + clang-tidy -p=build --export-fixes=diagnostics.yaml `cat cppfiles.txt` - name: Run clang-tidy-pr-comments action uses: deepmodeling/abacus-code-reviewer@master with: