Skip to content

Fix PHPUnit matrix dependency resolution for PHP 8.1–8.3 - #84

Merged
se7enxweb merged 3 commits into
mainfrom
copilot/fix-failing-github-actions-job
Aug 18, 2026
Merged

Fix PHPUnit matrix dependency resolution for PHP 8.1–8.3#84
se7enxweb merged 3 commits into
mainfrom
copilot/fix-failing-github-actions-job

Conversation

Copilot AI commented Aug 18, 2026

Copy link
Copy Markdown

The PHPUnit workflow was mutating the CI Composer manifest for older PHP matrix entries while still installing from a lockfile pinned to PHPUnit 13. As a result, the PHP 8.1 job failed during dependency installation before PHPUnit could run.

  • CI dependency resolution

    • Switch matrix entries that override phpunit/phpunit (8.1, 8.2, 8.3, and DB test jobs) from composer install to targeted composer update phpunit/phpunit --with-all-dependencies.
    • Keep the existing lockfile-based install path for matrix entries that do not rewrite the PHPUnit constraint (8.4, 8.5).
  • Behavioral impact

    • Aligns Composer’s operation with the generated composer.ci-phpunit.json contents for each matrix leg.
    • Prevents lockfile/constraint mismatches when CI downgrades PHPUnit for older PHP versions.
  • Workflow shape

    • Preserve the existing CI overlay approach and package set.
    • Limit the change to workflow logic; no production or test source changes.
if [[ -n "${PHPUNIT_VERSION:-}" ]]; then
  COMPOSER="${CI_COMPOSER_FILE}" composer update phpunit/phpunit \
    --with-all-dependencies --prefer-dist --no-progress --no-interaction \
    --ignore-platform-req=php --ignore-platform-req=ext-mongodb
else
  COMPOSER="${CI_COMPOSER_FILE}" composer install \
    --prefer-dist --no-progress --no-interaction \
    --ignore-platform-req=php --ignore-platform-req=ext-mongodb
fi

Copilot AI and others added 2 commits August 18, 2026 12:51
Co-authored-by: se7enxweb <51429274+se7enxweb@users.noreply.github.com>
Co-authored-by: se7enxweb <51429274+se7enxweb@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job PHPUnit (PHP 8.1) Fix PHPUnit matrix dependency resolution for PHP 8.1–8.3 Aug 18, 2026
Copilot AI requested a review from se7enxweb August 18, 2026 12:54
@se7enxweb
se7enxweb marked this pull request as ready for review August 18, 2026 12:55
@se7enxweb
se7enxweb merged commit ff07064 into main Aug 18, 2026
7 checks passed
@se7enxweb
se7enxweb deleted the copilot/fix-failing-github-actions-job branch August 18, 2026 12:55
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