I was lately playing with PHPStan and running it against ODM was also on my list :+1:
@alcaeus Can I PR it? :innocent:
@carusogabriel feel free, I'm in the middle of setting up new computer (and seeing if I can run MongoDB on that funky linux in Windows 10 :P )
As I'm new to PHPStan, just a question: by [2.0] Add PHPStan means require it in dev-dependencies and work on phpstan.neon?
My roadmap was:
Level 1 and parts of level 2 added in #1714.
@alcaeus @malarzm May we add https://github.com/phpstan/phpstan-strict-rules or better to increase PHPStan level first?
Problem with increasing levels is typehinting Common's metadata (as per interface) and expecting our own implementation. I'm :-1: for adding ifs but maybe assert would be the way to go?
Gonna try tonight to increase one more level :smile:
Closing, we're on level 7 for now :)
Most helpful comment
Problem with increasing levels is typehinting Common's metadata (as per interface) and expecting our own implementation. I'm :-1: for adding
ifs but maybeassertwould be the way to go?