phpstan icon indicating copy to clipboard operation
phpstan copied to clipboard

Breaking change!

Open hosni opened this issue 1 year ago • 4 comments

Hello, in the https://github.com/php-actions/phpstan/pull/21 there is a breaking change!
Many projects use the standalone version of PHPstan without using Composer.

But with changes that were made in https://github.com/php-actions/phpstan/pull/21 the default value of version is changed to composer and it's a breaking change! Because many CIs relied on this and it was broken now!
I think it should be published under a new version! Or, using a fallback strategy to fix this.

image

hosni avatar Dec 09 '24 20:12 hosni

Hi @hosni thank you for reporting this.

You are completely correct! I appologise for this oversight.

What course of action do you think it the most appropriate? Should I recall the changes to the latest rolling release, and make a new stable release version, or do you think it would cause more issues in altering the existing release?

g105b avatar Dec 12 '24 11:12 g105b

Hi @g105b thank for your great job here.

I think it's better to just change the default value of the version input to it's previous value (latest), and keep the changes still remain. There is no need to revert whole PR, just changing the default value is enough.

And in the next major version, change it back again to composer.

Anything missed?

hosni avatar Dec 15 '24 19:12 hosni

I think that's the best approach. I will make this change today, and mark the change to be made for the next version. Thanks for you insight.

g105b avatar Dec 19 '24 09:12 g105b

I can currently reproduce this issue with the following CI file: https://github.com/AniLeo/rpcs3-compatibility/blob/master/.github/workflows/master.yml

I have worked around the issue by specifying version: latest

AniLeo avatar Feb 08 '25 16:02 AniLeo