Skip to content

Adjustments to run Atlas.Query with PHP 8.5 - #23

Open
mikesoule wants to merge 1 commit into
atlasphp:2.xfrom
mikesoule:php85-compatibility
Open

mikesoule wants to merge 1 commit into
atlasphp:2.xfrom
mikesoule:php85-compatibility

Conversation

@mikesoule

Copy link
Copy Markdown

Follow up on #21 to run the tests and static analysis on PHP 8.5 while retaining PHP 8.0 support.

  • Changed implicitly nullable parameters in test helpers to explicitly accept null.
  • Added tests for getLastInsertId() with omitted and null names.
  • Upgraded PHPStan to ^1.0, matching Atlas.Pdo.
  • Added PHP 8.5 to both GitHub Actions matrices.
  • Added an unreleased changelog entry.

No production code or public method signatures are changed.

Verified tests and PHPStan on PHP 8.0, 8.1, 8.2, 8.3, 8.4, and 8.5. All versions pass 12 tests with 18 assertions, and PHPStan reports no errors.

Remove implicit nullable parameter deprecations from test helpers and
cover last insert IDs with omitted and null names. Upgrade PHPStan to
1.x and extend both CI matrices to PHP 8.5 while retaining PHP 8.0 support.
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