Skip to content

Run CI tests on PHP 8.4 instead of 8.2 - #23

Merged
basi merged 1 commit into
mainfrom
work/update-ci-to-php84
Aug 26, 2026
Merged

basi merged 1 commit into
mainfrom
work/update-ci-to-php84

Conversation

@basi

@basi basi commented Aug 26, 2026

Copy link
Copy Markdown
Owner

Summary

Update the CI test matrix to run on PHP 8.4 instead of PHP 8.2.

The codebase already ships PHP 8.4 fixes (changelog 2025-02-04, upstream kamilwylegala#83 merged as f9dc42f) and downstream consumers run this fork on PHP 8.4 in production, but CI has never actually exercised 8.4 — the highest version in the matrix was 8.2.

Changes

  • .github/workflows/tests.yml: matrix include 8.2 × mysql → 8.4 × mysql. Resulting matrix: 8.0 × (mysql, pgsql, sqlite) + 8.4 × mysql.
  • README.md: updated the stale "Github actions are running tests on PHP 8.0, 8.1." note to reflect the new matrix (it was already out of date vs. the actual 8.0/8.2 matrix).

Verification

  • PECL mcrypt 1.0.7+ supports PHP 8.4 (1.0.9 supports up to 8.6), so the setup-php extensions step (mcrypt included) installs cleanly on 8.4.
  • PHPUnit 9.6's convertDeprecationsToExceptions defaults to false, so new PHP 8.4 deprecation notices do not fail the suite.
  • On this PR, the Install PHP with extensions step runs before Composer and validates the PHP 8.4 toolchain immediately; full test verification is blocked by the pre-existing Composer failure below.

Known CI status

🤖 Generated with Claude Code

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@basi
basi marked this pull request as ready for review August 26, 2026 08:21
@basi
basi merged commit f4461f9 into main Aug 26, 2026
0 of 4 checks passed
@basi
basi deleted the work/update-ci-to-php84 branch August 26, 2026 08:21
@basi basi self-assigned this Aug 26, 2026
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.

1 participant