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 9, 2024
1 parent 408c7d7 commit 738afab
Showing 1 changed file with 21 additions and 8 deletions.
29 changes: 21 additions & 8 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 <info@kdab.com>
# SPDX-FileCopyrightText: 2024 Klarälvdalens Datakonsult AB, a KDAB Group company <info@kdab.com>
#
# SPDX-License-Identifier: MIT

Expand All @@ -11,31 +11,44 @@ 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
uses: aseprite/get-ninja@main
if: runner.os == 'Windows'
uses: turtlesec-no/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
run: >
cmake -S . -B ./build -G Ninja -DKDUTILS_BUILD_TESTS=True -DCMAKE_CXX_CLANG_TIDY="clang-tidy-18" \
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 738afab

Please sign in to comment.