Skip to content

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

Merged
dingo-d merged 1 commit into
developfrom
feature/ghactions-setup-php-fail-fast
Aug 4, 2026
Merged

GH Actions: fail "setup-php" if requested tooling could not be installed#2769
dingo-d merged 1 commit into
developfrom
feature/ghactions-setup-php-fail-fast

Conversation

@jrfnl

@jrfnl jrfnl commented Jul 26, 2026

Copy link
Copy Markdown
Member

Description

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

Suggested changelog entry

N/A

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 3.4.x milestone Jul 26, 2026
@jrfnl jrfnl modified the milestones: 3.4.1, 3.4.x Jul 27, 2026
@dingo-d
dingo-d merged commit 65a1350 into develop Aug 4, 2026
42 checks passed
@dingo-d
dingo-d deleted the feature/ghactions-setup-php-fail-fast branch August 4, 2026 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants