thatguygriffandClaude Opus 5
8122c158cf
CI / Tests (PHP 8.2) (pull_request) Successful in 51s
CI / No Debug Code (pull_request) Successful in 2s
CI / Coding Standards (pull_request) Successful in 2m54s
CI / PHPStan (pull_request) Successful in 2m55s
CI / Tests (PHP 8.3) (pull_request) Successful in 2m43s
CI / Build Plugin Zip (pull_request) Skipped
CI / Tests (PHP 8.1) (pull_request) Successful in 42s
`true` as a return type is PHP 8.2, but the plugin advertises 8.1, so the family screen's two service calls fataled on the 8.1 test job while every other job passed. They now return `?\WP_Error` — null on success — which matches RegistrationGate::validate() and works on 8.1. PHPStan was analysing against whatever PHP happened to be running (8.3 in CI, newer locally), so `composer lint` was green on syntax the plugin promises not to use. It is now pinned to the supported 8.1-8.3 range, which reproduces this failure at lint time instead of three jobs later. Co-Authored-By: Claude Opus 5 <[email protected]>
17 lines
535 B
Plaintext
17 lines
535 B
Plaintext
includes:
|
|
- vendor/szepeviktor/phpstan-wordpress/extension.neon
|
|
|
|
parameters:
|
|
level: 10
|
|
# Analyse against the whole supported range, not whatever PHP happens to be
|
|
# running. Without this, syntax newer than the `Requires PHP: 8.1` header
|
|
# promises passes lint on a modern local PHP and only fails in the 8.1 test
|
|
# job — which is how a PHP 8.2 `true` return type once reached CI.
|
|
phpVersion:
|
|
min: 80100
|
|
max: 80300
|
|
paths:
|
|
- src
|
|
bootstrapFiles:
|
|
- tests/bootstrap.php
|