From 60aa6ee4d08bcc585177bbfa8b88a91ea5a4cf1e Mon Sep 17 00:00:00 2001
From: James Titcumb <james@asgrim.com>
Date: Tue, 28 Dec 2021 10:53:26 +0000
Subject: [PATCH] Use fail-fast:true for shivammathur/setup-php

See:

https://github.com/shivammathur/setup-php/issues/548#issuecomment-1001646667
---
 .github/workflows/laravel.yml | 4 ++++
 .github/workflows/lumen.yml   | 4 ++++
 .github/workflows/quality.yml | 6 ++++++
 .github/workflows/symfony.yml | 2 ++
 .github/workflows/tests.yml   | 2 ++
 5 files changed, 18 insertions(+)

diff --git a/.github/workflows/laravel.yml b/.github/workflows/laravel.yml
index 07d605fa..49455dfb 100644
--- a/.github/workflows/laravel.yml
+++ b/.github/workflows/laravel.yml
@@ -40,6 +40,8 @@ jobs:
           php-version: "${{ matrix.php-version }}"
           tools: pecl, composer:v2.1
           extensions: ${{ matrix.extensions }}
+        env:
+          fail-fast: true
       # --no-update then a full `composer update` is needed to overcome locked dependencies
       # See: https://github.com/composer/composer/issues/9561
       - name: "Remove existing requirements components (avoid conflicts)"
@@ -87,6 +89,8 @@ jobs:
           php-version: "${{ matrix.php-version }}"
           tools: pecl, composer:v2.1
           extensions: ${{ matrix.extensions }}
+        env:
+          fail-fast: true
       - name: "Install Laravel quickstart project"
         run: "composer create-project laravel/laravel:${{ matrix.laravel-version}} test-app --prefer-dist"
       - name: "Add scout-apm-php as a repository"
diff --git a/.github/workflows/lumen.yml b/.github/workflows/lumen.yml
index 0bf87507..e0230d81 100644
--- a/.github/workflows/lumen.yml
+++ b/.github/workflows/lumen.yml
@@ -44,6 +44,8 @@ jobs:
           php-version: "${{ matrix.php-version }}"
           tools: pecl, composer:v2.1
           extensions: ${{ matrix.extensions }}
+        env:
+          fail-fast: true
       # --no-update then a full `composer update` is needed to overcome locked dependencies
       # See: https://github.com/composer/composer/issues/9561
       - name: "Remove existing requirements components (avoid conflicts)"
@@ -93,6 +95,8 @@ jobs:
           php-version: "${{ matrix.php-version }}"
           tools: pecl, composer:v2.1
           extensions: ${{ matrix.extensions }}
+        env:
+          fail-fast: true
       - name: "Install Lumen quickstart project"
         run: "composer create-project laravel/lumen:${{ matrix.lumen-version}} test-app --prefer-dist"
       - name: "Add scout-apm-php as a repository"
diff --git a/.github/workflows/quality.yml b/.github/workflows/quality.yml
index 4cad7585..ef88229a 100644
--- a/.github/workflows/quality.yml
+++ b/.github/workflows/quality.yml
@@ -20,6 +20,8 @@ jobs:
           coverage: "none"
           php-version: "8.0"
           tools: composer:v2.1
+        env:
+          fail-fast: true
       - name: "Install dependencies"
         run: "composer install"
       - name: "Run PHP_CodeSniffer"
@@ -36,6 +38,8 @@ jobs:
           coverage: "none"
           php-version: "8.0"
           tools: composer:v2.1
+        env:
+          fail-fast: true
       - name: "Install dependencies"
         run: "composer install"
       - name: "Run Psalm"
@@ -54,6 +58,8 @@ jobs:
           coverage: "none"
           php-version: "8.0"
           tools: composer:v2.1
+        env:
+          fail-fast: true
       - name: "Require Roave/BackwardCompatibilityCheck"
         run: "composer require --no-update --no-interaction --prefer-dist --prefer-stable --dev roave/backward-compatibility-check:^6.0.1"
       - name: "Composer update with new requirements"
diff --git a/.github/workflows/symfony.yml b/.github/workflows/symfony.yml
index e3f76fcd..6d31a5b6 100644
--- a/.github/workflows/symfony.yml
+++ b/.github/workflows/symfony.yml
@@ -40,6 +40,8 @@ jobs:
           php-version: "${{ matrix.php-version }}"
           tools: pecl, composer:v2.1
           extensions: ${{ matrix.extensions }}
+        env:
+          fail-fast: true
       # --no-update then a full `composer update` is needed to overcome locked dependencies
       # See: https://github.com/composer/composer/issues/9561
       - name: "Remove existing requirements components (avoid conflicts)"
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index 1713263b..fbfd2808 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -43,6 +43,8 @@ jobs:
           php-version: "${{ matrix.php-version }}"
           tools: pecl, composer:v2.1
           extensions: "curl, ${{ matrix.extensions }}"
+        env:
+          fail-fast: true
         # Normally, we'd just add "scoutapm" to the above extensions in shivammathur/setup-php, but libcurl appears to
         # be missing wherever the extension is built (not immediately obvious), so install it first
       - name: "Install scoutapm extension"