Skip to content

GH Actions: fail "setup-php" if requested tooling/extensions could not be installed - #5714

Merged
jordikroon merged 1 commit into
php:masterfrom
jrfnl:feature/ghactions-setup-php-fail-fast
Jul 28, 2026
Merged

GH Actions: fail "setup-php" if requested tooling/extensions could not be installed#5714
jordikroon merged 1 commit into
php:masterfrom
jrfnl:feature/ghactions-setup-php-fail-fast

Conversation

@jrfnl

@jrfnl jrfnl commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Setup-PHP will normally "gracefully" show a warning and not fail the build when an extension or tool failed to install.

In most cases, this is not particularly useful as that means that either there will be a failure later on in the build due to the extension or tool missing, or the build will not be representative of what is supposed to be run/tested.

This commit changes this behaviour to fail the build at the setup-php step, which also makes debugging these type of build failures much more straight-forward.

Ref: https://github.com/shivammathur/setup-php?tab=readme-ov-file#fail-fast-optional

…t be installed

Setup-PHP will normally "gracefully" show a warning and not fail the build when an extension or tool failed to install.

In most cases, this is not particularly useful as that means that either there will be a failure later on in the build due to the extension or tool missing, or the build will not be representative of what is supposed to be run/tested.

This commit changes this behaviour to fail the build at the `setup-php` step, which also makes debugging these type of build failures much more straight-forward.

Ref: https://github.com/shivammathur/setup-php?tab=readme-ov-file#fail-fast-optional
@jordikroon

Copy link
Copy Markdown
Member

Thank you. I didn't know about this flag. But good to have it.

@jordikroon
jordikroon merged commit 8e89023 into php:master Jul 28, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants