Skip to content

Commit

Permalink
Add clang tidy for more systems.
Browse files Browse the repository at this point in the history
  • Loading branch information
Alexhuszagh committed Sep 7, 2024
1 parent 762eda3 commit 2880e81
Showing 1 changed file with 13 additions and 4 deletions.
17 changes: 13 additions & 4 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,15 +44,24 @@ jobs:
scripts/lint.sh
lint-cpp:
runs-on: ubuntu-latest
strategy:
matrix:
os: [ubuntu-latest, windows-latest, macos-latest]
runs-on: ${{ matrix.os }}
steps:
- uses: actions/checkout@v4
- name: Install dependencies
run: |
sudo apt update
sudo apt install clang-tidy -y
python -m pip install --upgrade pip
pip install clang-tidy
- name: Analysing the code with clang-tidy
shell: bash
run: |
set -eux pipefail
clang-tidy -checks=-*,clang-analyzer-*,-clang-analyzer-cplusplus* -extra-arg=-std=c++17 example/breeze_theme.hpp --
# Windows oddly requires C++20 support due to internal bugs.
extra_arg="-std=c++17"
if [[ "$RUNNER_OS" == "Windows" ]]; then
extra_arg="-std=c++20"
fi
clang-tidy -checks=-*,clang-analyzer-*,-clang-analyzer-cplusplus* -extra-arg="${extra_arg}" example/breeze_theme.hpp --

0 comments on commit 2880e81

Please sign in to comment.