Skip to content

⚠️ Require php 8.5 and use its new language features - #37

Open
pavog wants to merge 1 commit into
masterfrom
require-php-8-5
Open

pavog wants to merge 1 commit into
masterfrom
require-php-8-5

Conversation

@pavog

@pavog pavog commented Nov 13, 2026

Copy link
Copy Markdown
Member

Codex required PHP 8.4 and CI tested against 8.4 and 8.5.
This PR drops PHP 8.4 support, which is a breaking change, and requires PHP 8.5.

Detective::detect() now uses array_first() to pick the best detection result after sorting, instead of reading index 0.

Changes

  • require php >=8.5 in composer.json and refresh the lock file
  • run ci only on php 8.5
  • use array_first() to pick the best detection result

@pavog pavog added this to the v5.0.0 milestone Nov 13, 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