- Issue created by @agentrickard
- π©πͺGermany webflo
webflo β changed the visibility of the branch 3458055-phpstan to hidden.
- π©πͺGermany webflo
webflo β changed the visibility of the branch 3458055-phpstan to active.
- π©πͺGermany webflo
webflo β changed the visibility of the branch 3458055-phpstan to hidden.
- π©πͺGermany webflo
webflo β changed the visibility of the branch 3458055-phpstan to active.
- πΊπΈUnited States agentrickard Georgia (US)
I think I am going to dial back to level 1 PHPStan for now.
- π©πͺGermany webflo
I think level 2 is achievable. We should create a baseline and refactor it step by step. Maybe one submodule after another.
- πΊπΈUnited States agentrickard Georgia (US)
Yes. I fixed my earlier problems and have been fixing things in reverse alphabetical order. Trying to finish up domain_config_ui today.
- First commit to issue fork.
- Status changed to Needs review
6 months ago 4:16pm 6 July 2024 - πΊπΈUnited States agentrickard Georgia (US)
This is all looking good. I have moved the remaining issues into phpstan-baseline.neon.
@webflo -- I think those can be fixed in a separate issue.
This could use a real-world test, but if it works for you, then I feel good about the state of the code and adding new maintainers.
- πΊπΈUnited States agentrickard Georgia (US)
We fixed everything except Drush, which should be a separate issue.
The attached patch did not work as documented here -- https://www.drush.org/12.x/dependency-injection/#create-method -- using Druah 12.4 on PHP 8.3.
That can be a separate issue.
-
webflo β
committed 877f662c on 2.0.x authored by
agentrickard β
Issue #3458055 by agentrickard, webflo: Configure and fix PHPStan issues
-
webflo β
committed 877f662c on 2.0.x authored by
agentrickard β
- Status changed to Fixed
5 months ago 12:33pm 16 July 2024 Automatically closed - issue fixed for 2 weeks with no activity.