Skip to content

Allow PHP 8.6 as max version and run CI on it - #6435

Merged
ondrejmirtes merged 1 commit into
2.2.xfrom
codex/allow-php-86-without-turbo
Sep 14, 2026
Merged

ondrejmirtes merged 1 commit into
2.2.xfrom
codex/allow-php-86-without-turbo

Conversation

@ondrejmirtes

@ondrejmirtes ondrejmirtes commented Sep 14, 2026

Copy link
Copy Markdown
Member

Allow phpVersion values through 80699 and cap newer Composer PHP constraints at PHP 8.6.99. Add PHP 8.6 to lint, tests, reflection golden tests, static analysis, and PHAR runtime tests, with updated version inference expectations.

Use version-specific expectations for filter_var_array(): PHP 8.6 narrows its declared return type from array|false|null to array|false. Keep all twelve dynamic-filter assertions for both version ranges.

Turbo builds and binary tests remain on PHP 8.3–8.5. The existing selector checks for the exact minor version's binary and skips Turbo on PHP 8.6 when no matching binary is shipped.

Supersedes #6403, squashed into one commit with the PHP 8.6 Turbo build attempts removed.

Validation: reproduced the PHP 8.6 assertion failure before fixing the expectations. The filter tests pass on PHP 8.5 and 8.6 using PHPUnit 12.5.35, and the adjacent filter_input_array() tests pass. Workflow lint and coding standards pass. The full local PHP 8.6 run is limited by missing cURL/OpenSSL extensions; CI runs with those extensions installed.

@ondrejmirtes
ondrejmirtes force-pushed the codex/allow-php-86-without-turbo branch from d57bcc5 to 153669c Compare September 14, 2026 07:26
@ondrejmirtes
ondrejmirtes force-pushed the codex/allow-php-86-without-turbo branch from 153669c to 65e6adb Compare September 14, 2026 07:28
@ondrejmirtes
ondrejmirtes merged commit c752b77 into 2.2.x Sep 14, 2026
240 of 242 checks passed
@ondrejmirtes
ondrejmirtes deleted the codex/allow-php-86-without-turbo branch September 14, 2026 07:29
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