@ciaranmcnulty I reached the max phpstan level, with only 15 errors added in the baseline. Most of them are related to case where we deal with string containing class names without defining their type as class-string, which is something that we can improve later. One of them is about ReflectionClass::isReadOnly being reported as undefined, which will go away once we run phpstan on PHP 8.2 (but as I have PHP 8.1 locally, it was easier that way for now). And 2 of them are false positives caused by https://github.com/phpstan/phpstan/issues/8439.
So this looks good enough for a merge IMO.
@ciaranmcnulty I reached the max phpstan level, with only 15 errors added in the baseline. Most of them are related to case where we deal with string containing class names without defining their type as
class-string, which is something that we can improve later. One of them is aboutReflectionClass::isReadOnlybeing reported as undefined, which will go away once we run phpstan on PHP 8.2 (but as I have PHP 8.1 locally, it was easier that way for now). And 2 of them are false positives caused by https://github.com/phpstan/phpstan/issues/8439. So this looks good enough for a merge IMO.