- Issue created by @dpi
- Merge request !10172Resolve PHPStan-Strict errors: staticMethod.nameCase β (Closed) created by dpi
- πΊπΈUnited States smustgrave
Since phpstan-strict isn't in core should we be fixing this right now or work to get phpstan-strict into core first?
- π¦πΊAustralia dpi Perth, Australia
@smustgrave the issue will come for that, these batch of issues that I've created will allow us to add it without increasing the baseline.
There isn't a lot of them, and its easier to solve upfront.
- π¦πΊAustralia dpi Perth, Australia
@smustgrave created an outline at π Resolve issues exposed by PHPStan Strict Active
- πΊπΈUnited States smustgrave
So brought this up in slack and think these should be postponed till itβs agreed that phpstan strict will be added. Didnβt see a ticket for that discussion
- πΊπΈUnited States smustgrave
Only temporarily postponing pending feedback on π Resolve issues exposed by PHPStan Strict Active
If we decide there no harm in merging these we can put back into review and I'll put top of my list to review. Just want to make sure they'll be accepted first.
- π¦πΊAustralia mstrelan
I added
checkFunctionNameCase: true
to the existing phpstan config and got the same result. As per #3487218-7: Resolve PHPStan-Strict errors: class.nameCase β we should fix these with or without phpstan enforcement, as they are clearly incorrect.Have updated the IS with steps to reproduce not requiring the strict package.
-
larowlan β
committed 2b9242c5 on 11.x
Issue #3487239 by dpi, smustgrave, mstrelan: Resolve PHPStan-Strict...
-
larowlan β
committed 2b9242c5 on 11.x
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
Committed to 11.x - thanks!
- Issue was unassigned.
- Status changed to Fixed
3 months ago 2:29am 17 January 2025 Automatically closed - issue fixed for 2 weeks with no activity.