Skip to content

Commit

Permalink
Use fail-fast:true for shivammathur/setup-php
Browse files Browse the repository at this point in the history
  • Loading branch information
asgrim committed Dec 28, 2021
1 parent 83af1cf commit 8fdf574
Show file tree
Hide file tree
Showing 5 changed files with 18 additions and 0 deletions.
4 changes: 4 additions & 0 deletions .github/workflows/laravel.yml
Expand Up @@ -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)"
Expand Down Expand Up @@ -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"
Expand Down
4 changes: 4 additions & 0 deletions .github/workflows/lumen.yml
Expand Up @@ -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)"
Expand Down Expand Up @@ -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"
Expand Down
6 changes: 6 additions & 0 deletions .github/workflows/quality.yml
Expand Up @@ -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"
Expand All @@ -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"
Expand All @@ -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.x-dev roave/better-reflection:5.0.x-dev"
- name: "Composer update with new requirements"
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/symfony.yml
Expand Up @@ -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)"
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/tests.yml
Expand Up @@ -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"
Expand Down

0 comments on commit 8fdf574

Please sign in to comment.