Skip to content

Commit

Permalink
Add clang-tidy on Windows on GitHub Actions
Browse files Browse the repository at this point in the history
Action turtlesec-no/get-ninja is used instead of aseprite/get-ninja as
the latter one puts ninja in some place that makes it not discoverable
for CMake in later steps.
  • Loading branch information
MiKom committed Jul 10, 2024
1 parent 408c7d7 commit 9b07e4f
Showing 1 changed file with 25 additions and 7 deletions.
32 changes: 25 additions & 7 deletions .github/workflows/nightly.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# SPDX-FileCopyrightText: 2023 Klarälvdalens Datakonsult AB, a KDAB Group company <[email protected]>
# SPDX-FileCopyrightText: 2024 Klarälvdalens Datakonsult AB, a KDAB Group company <[email protected]>
#
# SPDX-License-Identifier: MIT

Expand All @@ -11,31 +11,49 @@ on:
push:

env:
CC: "/usr/bin/clang"
CXX: "/usr/bin/clang++"
CC: "clang"
CXX: "clang++"

jobs:
build:
runs-on: ubuntu-24.04
runs-on: ${{ matrix.os }}
strategy:
matrix:
include:
- name: Ubuntu
os: ubuntu-24.04
# Specify executable name on Linux to have control over the version.
# Sadly, on Linux these per-version executables aren't there.
tidy-executable: clang-tidy-18
- name: Windows
os: windows-2022
tidy-executble: clang-tidy

steps:
- name: Checkout sources
uses: actions/checkout@v4

- name: Install ninja-build tool
if: runner.os == 'Windows'
uses: aseprite/get-ninja@main

- name: Install linux dependencies
if: runner.os == 'Linux'
run: |
sudo apt update -qq
sudo apt install -y libxkbcommon-dev libxcb-xkb-dev \
libxkbcommon-x11-dev wayland-scanner++ wayland-protocols \
libwayland-dev xvfb
libwayland-dev xvfb ninja-build
- name: Configure project
if: runner.os == 'Windows'
run: >
cmake -S . -B ./build -G Ninja -DKDUTILS_BUILD_TESTS=True -DCMAKE_CXX_CLANG_TIDY="clang-tidy-18" \
echo $env:GITHUB_PATH
cat $env:GITHUB_PATH
cmake -S . -B ./build -G Ninja -DKDUTILS_BUILD_TESTS=True -DCMAKE_CXX_CLANG_TIDY="${{matrix.tidy-executable}}"
-DCMAKE_BUILD_TYPE=Release
- name: Build Project
run: cmake --build ./build
run: cmake --build ./build

0 comments on commit 9b07e4f

Please sign in to comment.