Make result-cache independent from configs include order

[staabm]

closes phpstan/phpstan#8778

tested via phpstan/phpstan#9857

[ondrejmirtes]

Thank you!

[staabm]

@ondrejmirtes any plans for a new release? I think this change will save us minutes on every commit in CI

[ondrejmirtes]

You don't have to hypothesize, you can verify it by requiring 1.10.x-dev 😊 Please report back the numbers.

[staabm]

if my testing in CI works as itended I can see PHPStan now using the cache, while it didn't before.
(I guess because of the extension loading order in our complex setup)

this means instead of a full project analysis which takes ~5min, it now only takes ~30secs because of cache utilization per commit.
(several phpstan invocations involved, because of a monorepo)

[ondrejmirtes]

Rector using result cache doesn't make any sense.

[staabm]

I had a typo in my above comment. I updated it to say "PHPStan" not rector" ;)

[ondrejmirtes]

Alright, can you please test the dev version anyway?

[staabm]

Alright, can you please test the dev version anyway?

I did. thats the numbers for the dev version

[ondrejmirtes]

Alright, thank you 😊

[staabm]

for reference - full project analysis on my local machine of the same project

$ vendor/bin/phpstan --version
PHPStan - PHP Static Analysis Tool 1.10.x-dev@19801d9

$ time composer phpstan
...
real    2m47.226s
user    0m0.015s
sys     0m0.000s

vs.

$ vendor/bin/phpstan --version
PHPStan - PHP Static Analysis Tool 1.10.33

$ time composer phpstan
...
real    2m40.789s
user    0m0.000s
sys     0m0.030s