Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
PHPStan 1.9.0 is out.
Steps to reproduce
Proposed resolution
Remaining tasks
User interface changes
API changes
Data model changes
Release notes snippet
Comment | File | Size | Author |
---|---|---|---|
#4 | 3319112-4.patch | 2.3 KB | longwave |
|
Comments
Comment #2
longwaveComment #3
mondrake1.9.1 is out already...
Also,
I do not think we need to bump the constraint if the baseline file remains unchanged.
Comment #4
longwaveRemoved composer.json change, updated to 1.9.1.
Comment #5
longwaveComment #6
mondrakelgtm
Comment #8
mondrakerandom fail?
Comment #10
catchCommitted/pushed to 10.1.x and cherry-picked to 10.0.x, thanks!
Comment #11
catchEven though this commit itself went OK, I don't seem to be able to do a full phpstan run on 1.9.1, looks like this:
(then a dozen or so similar errors).
If I roll back to the commit before this one, it's OK. If I run phpstan inside a ddev vm, it's also fine. I'm using stock Ubuntu LTS PHP 8.1 on my actual local cli, so I reckon it might be that.
Technically we don't support Ubuntu's stock PHP 8.1 install because it's 8.1.2, but #3307248: Special case Ubuntu PHP versions for some system_requirements() warnings would change that (which is, apart from laziness, why I kept running it), so re-opening in case there's something we need to do here.
Comment #12
catchTalked with @alexpott, this is as far as we got:
Ubuntu LTS PHP 8.1.2 -
My ddev container:
Alex has no problems running phpstan, and his PCRE version is also 10.40, so maybe it's an issue with 10.39?
Comment #13
mondrakeSounds like #11 and #12 are oos here and hold this issue unnecessarily.