- Issue created by @spokje
- Merge request !7297$ vendor/bin/phpstan analyze --configuration=core/phpstan.neon.dist... → (Closed) created by spokje
- Status changed to Needs review
9 months ago 8:13am 3 April 2024 - 🇳🇱Netherlands spokje
Note: Creating a new PHPStan-baseline exposed some formatting differences between the generated and committed-in-HEAD one.
Let's use this issue to reconcile these.As a side note: If people are manually editing the baseline, let's make the GitLab job fail on these differences, so we don't need this kind of reconciliation.
- Status changed to Needs work
9 months ago 8:34am 3 April 2024 - 🇬🇧United Kingdom longwave UK
Let's bump the minimum version of
mglaman/phpstan-drupal
here, because it fixes a PHPStan crash over in 📌 Use a tagged service iterator for uninstall validators instead of individual lazy proxies Needs review - Status changed to Needs review
9 months ago 8:53am 3 April 2024 - 🇳🇱Netherlands spokje
Ah crap, of course, forgot that in my "rage" about the baseline formatting.
- Status changed to RTBC
9 months ago 9:33am 3 April 2024 - Status changed to Fixed
9 months ago 10:37am 3 April 2024 - 🇬🇧United Kingdom alexpott 🇪🇺🌍
-
alexpott →
committed 1e955e9e on 10.3.x
Issue #3437778 by Spokje, longwave: Bump phpstan/phpstan and mglaman/...
-
alexpott →
committed 1e955e9e on 10.3.x
-
alexpott →
committed 8fe4fb06 on 11.x
Issue #3437778 by Spokje, longwave: Bump phpstan/phpstan and mglaman/...
-
alexpott →
committed 8fe4fb06 on 11.x
Automatically closed - issue fixed for 2 weeks with no activity.