- Issue created by @timohuisman
- 🇳🇱Netherlands timohuisman Leiden, Netherlands
I've created a MR so we can see if the tests are still green.
- Status changed to Needs review
4 months ago 2:40pm 24 April 2025 - First commit to issue fork.
- 🇩🇪Germany jurgenhaas Gottmadingen
I've updated the MR to also allow
phpstan/phpstan-deprecation-rules
version 2 because otherwise, we would still get PhpStan version 1.I've tested the MR locally and it now works by pulling the latest PhpStan version and also the latest mglaman/phpstan-drupal
Leaving the status at NR as I've changed the MR, so I can't move it to RTBC.
- 🇳🇱Netherlands kingdutch
Based on the code changes I'd be happy to RTBC this, but the tests are failing so I think this needs some work to figure out why that is.
- 🇩🇪Germany jurgenhaas Gottmadingen
I've rebased to the latest changes in the dev branch, let's see if the tests are passing now.
- 🇩🇪Germany jurgenhaas Gottmadingen
Back to NR, the pipelines are all green now.
- 🇳🇱Netherlands kingdutch
🎉
Looks good to me, ensures that Upgrade Status can be installed for projects that got ahead of it with PHPStan and looks to work as expected :)
- 🇧🇾Belarus f1mishutka Minsk
Nice and simple patch, thanks!
Please merge it because it is a sad show stopper for switching to Drupal 11.2
- Status changed to RTBC
2 months ago 11:20pm 23 June 2025 - 🇨🇦Canada joelpittet Vancouver
This caught me trying to upgrade to D11 and then got held back from Drupal 11.2. +1 for merge
- 🇧🇾Belarus f1mishutka Minsk
A kind of funny situation: Module supposed to be first and must-have helper for Drupal upgrades is a show stopper for Drupal 11.2 update.
Please merge this as soon as possible...
- 🇭🇺Hungary Gábor Hojtsy Hungary
gábor hojtsy → made their first commit to this issue’s fork.
-
gábor hojtsy →
committed c7535c0f on 4.x authored by
timohuisman →
Issue #3499888: Add support for phpstan/phpstan:2.0.0
-
gábor hojtsy →
committed c7535c0f on 4.x authored by
timohuisman →
- 🇭🇺Hungary Gábor Hojtsy Hungary
Thanks all! I get the irony! :) Hope this helps. Will do a release too later today, but will check first if there are other low hanging fruits to include too.
- 🇭🇺Hungary Gábor Hojtsy Hungary
None of the fruits seemed low enough so released this with one other commit I already had :D https://www.drupal.org/project/upgrade_status/releases/4.3.8 → Thanks again!
Automatically closed - issue fixed for 2 weeks with no activity.