Skip to content

Commit

Permalink
build: run linter on all files on main
Browse files Browse the repository at this point in the history
  • Loading branch information
Taepper committed Jan 14, 2025
1 parent 2dc6af5 commit cae1ffe
Showing 1 changed file with 18 additions and 1 deletion.
19 changes: 18 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -188,7 +188,7 @@ jobs:
run: cd endToEndTests && SILO_URL=localhost:8080 npm run test

linterChanges:
name: Build And Run linter
name: Build/Run linter on changed files
needs: dockerImageUnitTests
if: github.event_name == 'pull_request'
runs-on: ubuntu-latest
Expand Down Expand Up @@ -230,3 +230,20 @@ jobs:
cmake --build build/Debug --target ${file%.cpp}.o
fi
done
linterAll:
name: Build/Run linter on all files
needs: dockerImageUnitTests
if: github.event_name == 'push'
runs-on: ubuntu-latest
container:
image: ghcr.io/genspectrum/lapis-silo-dependencies:commit-${{ github.sha }}
steps:
- uses: actions/checkout@v4

- shell: bash
name: Configure and run clang-tidy on changed files
run: |
mv /src/build .
cmake -DBUILD_WITH_CLANG_TIDY=on -D CMAKE_BUILD_TYPE=Debug -B build/Debug
cmake --build build/Debug

0 comments on commit cae1ffe

Please sign in to comment.