Skip to content

ci: fix the Behat workflow setup - #116

Merged
mrrobot47 merged 3 commits into
EasyEngine:developfrom
mrrobot47:ci/fix-behat-workflow
Sep 30, 2026
Merged

mrrobot47 merged 3 commits into
EasyEngine:developfrom
mrrobot47:ci/fix-behat-workflow

Conversation

@mrrobot47

@mrrobot47 mrrobot47 commented Sep 30, 2026 •

Copy link
Copy Markdown
Member

The Behat jobs have been failing at setup because actions/cache@v2 is deprecated.

This PR:

  • bumps actions/cache to v4;
  • makes the downloaded docker-compose binary executable;
  • builds the phar without dev autoloading;
  • drops PHP 7.2 and 7.3 from the matrix. PHP 7.2 couldn't run the test dependencies, and neither version is supported any more (core requires PHP >= 7.4 and its CI doesn't test them).

Because of the matrix change, the required status checks need Behat Tests - PHP 7.2 and Behat Tests - PHP 7.3 removed; branch protection now requires WIP, Behat Tests - PHP 7.4 and Behat Tests - PHP 8.0.

Earlier run on the fork, before the matrix change (7.3, 7.4 and 8.0 pass; 7.2 fails on the dependency parse error): https://github.com/mrrobot47/site-type-php/actions/runs/36710563782

PHP 7.2 and 7.3 are no longer supported: the core README requires PHP >= 7.4 and core CI does not test them.
@rtBot
rtBot requested a balanced review from Copilot September 30, 2026 12:18

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The focused workflow changes are consistent with the stated purpose and supported by successful PHP 7.4 and 8.0 runs.

Review effort: Balanced
Findings: None

@mrrobot47 mrrobot47 changed the title ci: fix the Behat workflow setup ci: fix the Behat workflow setup Sep 30, 2026
@mrrobot47 mrrobot47 changed the title ci: fix the Behat workflow setup ci: fix the Behat workflow setup Sep 30, 2026
@mrrobot47
mrrobot47 merged commit f94e0a5 into EasyEngine:develop Sep 30, 2026
3 of 5 checks passed
@mrrobot47
mrrobot47 deleted the ci/fix-behat-workflow branch September 30, 2026 12:52
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