- Issue created by @darvanen
- First commit to issue fork.
- @sime opened merge request.
- Status changed to Needs work
7 months ago 7:09am 12 April 2024 - 🇦🇺Australia sime Melbourne
Notes
Phpstan couldn't find the base class in core (because it was removed in drupal 10). I see this module supports D10 so I just removed the class.
One issue i haven't been able to resolve yet.
- Status changed to Needs review
7 months ago 2:41am 13 April 2024 - 🇦🇺Australia sime Melbourne
When testing against next version (ie D11) PHPstan fails because a CKEditor 4 related class dependency no longer exists.
In this discussion it was suggested we haven't quite solved the best way to do this.
I have removed the "next version" test in this PR for now (so that is it all green), and happy for further direction - ie would you accept a bunch of stubs to make this work? Or do you want to wait for some other solutions to emerge.
Note that if just the PHPstan NEXT_MAJOR is "warning" (orange) the project page would still show that CI is "pass" (green).
- Status changed to Needs work
7 months ago 4:04am 13 April 2024 - 🇦🇺Australia darvanen Sydney, Australia
#5: The class was moved to a contrib module but that module is no longer supported either, so yes, removing that class is best, thanks.
#6: I'm happy for next_major to be orange if the project page is green.
There's a bunch of conflicts now that 📌 PHPCS - enable and fix Fixed is in, I wasn't expecting that. Do you want to fix the merge conflict or should I have a go?
- Status changed to Needs review
7 months ago 9:27am 13 April 2024 - 🇦🇺Australia sime Melbourne
Ok that's ready with the one orange. I'd be interested to see what the status shows when that is merged.
- 🇦🇺Australia darvanen Sydney, Australia
Here goes nothing, and if it does remain orange I'll just turn off next_major with a postponed follow-up until it's resolved.
- Status changed to RTBC
7 months ago 6:37am 14 April 2024 -
darvanen →
committed 2b23f38b on 2.x authored by
sime →
Issue #3433140 by sime, darvanen: PHPStan - enable and fix
-
darvanen →
committed 2b23f38b on 2.x authored by
sime →
- Status changed to Fixed
7 months ago 6:50am 14 April 2024 Automatically closed - issue fixed for 2 weeks with no activity.