[PR #9314] Run static analysis with language level PHP 8.1 #11517

Closed
opened 2026-01-22 16:11:01 +01:00 by admin · 0 comments
Owner

Original Pull Request: https://github.com/doctrine/orm/pull/9314

State: closed
Merged: Yes


Continuation of #9310

While PHP 7.1 is the lowest supported version, the repository contains features that require a more recent PHP version and we are currently hardly able to run static analysis on those parts.

Letting the static analysis tools operate on PHP 7.1 as language level causes unnecessary pain and barely spots real issues. Especially Psalm has a history of reporting funny errors if the dependencies are on a higher language level. I'd suggest to let the test suite cover PHP 7.1 support and let the SA tools operate on PHP 8.1 instead.

**Original Pull Request:** https://github.com/doctrine/orm/pull/9314 **State:** closed **Merged:** Yes --- Continuation of #9310 While PHP 7.1 is the lowest supported version, the repository contains features that require a more recent PHP version and we are currently hardly able to run static analysis on those parts. Letting the static analysis tools operate on PHP 7.1 as language level causes unnecessary pain and barely spots real issues. Especially Psalm has a history of reporting funny errors if the dependencies are on a higher language level. I'd suggest to let the test suite cover PHP 7.1 support and let the SA tools operate on PHP 8.1 instead.
admin added the pull-request label 2026-01-22 16:11:01 +01:00
admin closed this issue 2026-01-22 16:11:01 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#11517