Skip to content

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

Merged
jrfnl merged 1 commit into
developfrom
feature/ghactions-setup-php-fail-fast
Oct 25, 2026
Merged

jrfnl merged 1 commit into
developfrom
feature/ghactions-setup-php-fail-fast

Conversation

@jrfnl

@jrfnl jrfnl commented Jul 26, 2026

Copy link
Copy Markdown
Collaborator

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 tested.

This commit changes this behaviour to fail select builds 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

@jrfnl jrfnl added this to the 1.4.x Next milestone Jul 26, 2026
@jrfnl
jrfnl requested a review from grogy July 26, 2026 21:38
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 tested.

This commit changes this behaviour to fail select builds 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
@jrfnl
jrfnl force-pushed the feature/ghactions-setup-php-fail-fast branch from c62db3a to bab8b5d Compare October 25, 2026 21:59
@jrfnl

jrfnl commented Oct 25, 2026

Copy link
Copy Markdown
Collaborator Author

Unilaterally deciding to merge this as waiting three months for a review for something straight-forward like this is getting silly.

@jrfnl
jrfnl merged commit 7ca416b into develop Oct 25, 2026
20 checks passed
@jrfnl
jrfnl deleted the feature/ghactions-setup-php-fail-fast branch October 25, 2026 22:01
@jrfnl jrfnl mentioned this pull request Oct 25, 2026
33 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant