Skip to content

Commit

Permalink
GH Actions: selectively use fail-fast with setup-php (#240)
Browse files Browse the repository at this point in the history
* GH Actions: selectively use `fail-fast` with setup-php

I've seen some recent build failures due to the `setup-php` action running into a rate limit and not downloading the required version of Composer. In the case of this action, that would make the test runs worthless.

The `setup-php` action runner defaults to _showing_ these type errors in the logs, but not stopping the workflow run.

So, specifically for those jobs where the Composer version is important, I'm adding the `fail-fast` option to `setup-php` to fail the build if the action runner ran into any errors.

Ref: https://github.com/shivammathur/setup-php#fail-fast-optional

* GH Actions: update PHP version for `composer-normalize`

Looks like the `composer-normalize` package has dropped support for PHP < 8.0 as of version `2.29.0`.

Refs:
* https://github.com/ergebnis/composer-normalize/releases/tag/2.29.0
* ergebnis/composer-normalize#998

Co-authored-by: jrfnl <jrfnl@users.noreply.github.com>
  • Loading branch information
jrfnl and jrfnl committed Jan 2, 2023
1 parent 83af392 commit 110668c
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion .github/workflows/continuous-integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ jobs:
- name: "Set up PHP"
uses: "shivammathur/setup-php@v2"
with:
php-version: 7.4
php-version: 8.1
tools: composer-normalize
coverage: none
- name: "Run Composer normalize"
Expand All @@ -47,6 +47,8 @@ jobs:
php-version: "latest"
tools: "composer:${{ matrix.composer-version }}"
coverage: "none"
env:
fail-fast: true
- name: "Run expect tests"
run: "composer test"

Expand Down Expand Up @@ -122,6 +124,8 @@ jobs:
php-version: "latest"
tools: "composer:${{ matrix.composer-version }}"
coverage: "none"
env:
fail-fast: true

- name: "Test: plain install"
uses: ./
Expand Down Expand Up @@ -251,6 +255,8 @@ jobs:
php-version: "latest"
tools: "composer:${{ matrix.composer-version }}"
coverage: "none"
env:
fail-fast: true

- name: "Test: plain install"
uses: ./
Expand Down

0 comments on commit 110668c

Please sign in to comment.