Skip to content

ci: fix the Behat workflow setup - #508

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-command/actions/runs/36710558552

PHP 7.2 and 7.3 are no longer supported: the core README requires PHP >= 7.4 and core CI does not test them.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 CI changes are consistent and the supported matrix passed in the referenced workflow run.

Review effort: Balanced
Findings: None

What changed in this PR

Updates the Behat CI workflow to restore supported test execution.

Changes:

  • Tests only PHP 7.4 and 8.0.
  • Upgrades Composer caching and fixes docker-compose permissions.
  • Excludes development autoloading from the PHAR build.
File Description
.github/​workflows/​test_and_build.yml Modernizes and repairs Behat workflow setup.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@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 8745529 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