CI fix

Alexhuszagh · Sep 7, 2024 · a9a2180 · a9a2180
1 parent 2880e81
commit a9a2180
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 6 deletions.
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -60,8 +60,11 @@ jobs:
         set -eux pipefail
 
         # 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"
+        if [[ "${RUNNER_OS}" == "Windows" ]]; then
+          extra_args="-extra-arg=-std=c++20"
+        elif [[ "${RUNNER_OS}" == "macOS" ]]; then
+          extra_arg="-extra-arg=-std=c++17 -extra-arg=--stdlib=libc++"
+        else
+          extra_args="-extra-arg=-std=c++17"
         fi
-        clang-tidy -checks=-*,clang-analyzer-*,-clang-analyzer-cplusplus* -extra-arg="${extra_arg}" example/breeze_theme.hpp --
+        clang-tidy -checks=-*,clang-analyzer-*,-clang-analyzer-cplusplus* ${extra_args} example/breeze_theme.hpp --
diff --git a/.github/workflows/theme.yml b/.github/workflows/theme.yml
@@ -19,8 +19,8 @@ jobs:
       shell: bash
       run: |
         python -m pip install --upgrade pip
-        if [[ "$RUNNER_OS" == "Windows" ]]; then
-            python -m pip install winrt-Windows.UI.ViewManagement winrt-Windows.UI
+        if [[ "${RUNNER_OS}" == "Windows" ]]; then
+          python -m pip install winrt-Windows.UI.ViewManagement winrt-Windows.UI
         fi
     - name: Checking our Python imports.
       run: |