fix: replace if check with proper matrix exclusion

LaboratoireMecaniqueLille · May 2, 2024 · f1a8e3a · f1a8e3a
1 parent 618d6a7
commit f1a8e3a
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/.github/workflows/test_python_package.yml b/.github/workflows/test_python_package.yml
@@ -24,10 +24,10 @@ jobs:
         python-version: ["3.7", "3.8", "3.9", "3.10", "3.11", "3.12"]
         # Run on all the supported platforms
         os: [ubuntu-latest, windows-latest, macos-latest]
-    # Cannot run for Python 3.7 on macOS as it is no longer supported
-    if: |
-      contains(fromJSON('["Linux", "Windows"]'), runner.os) ||
-      contains(fromJSON('["3.8", "3.9", "3.10", "3.11", "3.12"]'), matrix.python-version)
+        # Cannot run for Python 3.7 on macOS as it is no longer supported
+        exclude:
+          - os: macos-latest
+            python-version: 3.7
 
     steps:
     # Checkout the repository