Add clang-tidy github action

awslabs · Jul 1, 2024 · 666dba6 · 666dba6
1 parent f6e499d
commit 666dba6
Showing 1 changed file with 10 additions and 4 deletions.
diff --git a/.github/workflows/build-and-test-linux.yml b/.github/workflows/build-and-test-linux.yml
@@ -121,6 +121,12 @@ jobs:
           sudo apt-get install -y intel-oneapi-compiler-dpcpp-cpp \
                                   intel-oneapi-compiler-fortran
 
+      - name: Install clang-tidy for clang compiler
+        if: matrix.compiler == 'clang'
+        run: |
+          sudo apt-get install -y clang-tidy
+
+
       - name: Install math libraries (OpenBLAS)
         if: matrix.math-libs == 'openblas'
         run: |
@@ -187,8 +193,7 @@ jobs:
           [[ "${{ matrix.with-eigensolver }}" == 'arpack' ]] && WITH_ARPACK='ON' || WITH_ARPACK='OFF'
 
           # Build and install (with unit tests)
-          mkdir palace-build && cd palace-build
-          cmake .. \
+          cmake -S . -B palace-build \
             -DCMAKE_INSTALL_PREFIX=$(pwd)/../palace-install \
             -DBUILD_SHARED_LIBS=$BUILD_SHARED \
             -DPALACE_WITH_64BIT_INT=$WITH_INT64 \
@@ -197,8 +202,9 @@ jobs:
             -DPALACE_WITH_STRUMPACK=$WITH_STRUMPACK \
             -DPALACE_WITH_MUMPS=$WITH_MUMPS \
             -DPALACE_WITH_SLEPC=$WITH_SLEPC \
-            -DPALACE_WITH_ARPACK=$WITH_ARPACK
-          make -j$NUM_PROC_BUILD palace-tests
+            -DPALACE_WITH_ARPACK=$WITH_ARPACK \
+            -DANALYZE_SOURCES_CLANG_TIDY=ON
+          cmake --build palace-build --parallel $NUM_PROC_BUILD --target palace-tests
 
       - name: Run unit tests
         env: