rectorphp / rector-symfony

Rector upgrade rules for Symfony
http://getrector.com
MIT License
180 stars 86 forks source link

Deprecate Symfony level sets #572

Closed TomasVotruba closed 8 months ago

TomasVotruba commented 8 months ago

It's not the best practice to use UP_TO_* sets in rector.php longer than is their single-run. They run dozens of rules with very complex logic, as it take to get from Symfony 2 to 7. It's like running a Tesla control and checking all elements available since the first horse ride :)

See section 3: https://getrector.com/blog/5-common-mistakes-in-rector-config-and-how-to-avoid-them

It's best practice - and we use it this way since the first Symfony rule set - to run those just once and remove .


Instead use only the latest major Symfony sets, to keep up to date and get fast and reliable Rector run :+1:

Thank you


TomasVotruba commented 8 months ago

See https://github.com/rectorphp/rector-src/pull/5477