CI Update runners in the regression-tests.yml

Author: jarzec

.github/workflows/regression-tests.yml .github/workflows/regression-tests.yaml.github/workflows/regression-tests.yml renamed to .github/workflows/regression-tests.yaml

@@ -9,6 +9,15 @@ on:
       - 'docs/**'
   workflow_dispatch:
 
+permissions:
+  # Restrict permissions to read-only for security
+  contents: read
+
+concurrency:
+  # Specific group name to cancel in-progress builds for the same PR/ref
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
 jobs:
   regression-tests:
     name: ${{ matrix.shortosname }} | ${{ matrix.compiler }} | ${{ matrix.cxx_std }} | ${{ matrix.stdlib }} | ${{ matrix.os }}
@@ -50,26 +59,17 @@ jobs:
             compiler: clang++
             cxx_std: c++2b
             stdlib: default
-          - os: macos-13
-            shortosname: mac-13
-            compiler: clang++
-            cxx_std: c++2b
-            stdlib: default
-          - os: macos-13
-            shortosname: mac-13
-            compiler: clang++-15
-            cxx_std: c++2b
-            stdlib: default
-          - os: windows-2022
-            shortosname: win-22
-            compiler: cl.exe
-            cxx_std: c++latest
-            stdlib: default
-          - os: windows-2022
-            shortosname: win-22
-            compiler: cl.exe
-            cxx_std: c++20
-            stdlib: default
+          # Comment out Windows tests until build on Windows is fixed
+          # - os: windows-2025
+          #   shortosname: win-25
+          #   compiler: cl.exe
+          #   cxx_std: c++latest
+          #   stdlib: default
+          # - os: windows-2025
+          #   shortosname: win-25
+          #   compiler: cl.exe
+          #   cxx_std: c++20
+          #   stdlib: default
     steps:
     - name: Checkout repo
       uses: actions/checkout@v4

regression-tests/run-tests.sh

@@ -154,17 +154,10 @@ else
 
     compiler_version=$("$cxx_compiler" --version)
 
-    if [[ "$compiler_version" == *"Apple clang version 14.0"* ||
-          "$compiler_version" == *"Homebrew clang version 15.0"* ]]; then
-        exec_out_dir="$expected_results_dir/apple-clang-14"
-        # We share the expected results dir for these two compilers, but there is one
-        # test which (as expected) fails to compile on both compilers, but has a slightly
-        # different error diagnostic because the clang path differs. So we exclude it from
-        # running. The alternative would be to duplicate the expected results files, which
-        # seems wasteful for just one test (that doesn't even compile).
-        exclude_test_filter="pure2-expected-is-as.cpp2"
-    elif [[ "$compiler_version" == *"Apple clang version 15.0"* ]]; then
+    if [[ "$compiler_version" == *"Apple clang version 15.0"* ]]; then
         exec_out_dir="$expected_results_dir/apple-clang-15"
+    elif [[ "$compiler_version" == *"Apple clang version 17.0"* ]]; then
+        exec_out_dir="$expected_results_dir/apple-clang-17"
     elif [[ "$compiler_version" == *"clang version 12.0"* ]]; then 
         exec_out_dir="$expected_results_dir/clang-12"
     elif [[ "$compiler_version" == *"clang version 15.0"* ]]; then 
@@ -216,8 +209,9 @@ else
     printf "\ncompiler_cmd: $compiler_cmd\n\n"
 fi
 
+printf "Full compiler version for '$cxx_compiler':\n$compiler_version\n\n"
+
 if [[ -d "$exec_out_dir" ]]; then
-    printf "Full compiler version for '$cxx_compiler':\n$compiler_version\n\n"
 
     printf "Directory with reference compilation/execution files to use:\n$exec_out_dir\n\n"
 else