driftingly/rector-laravel

Create sets for laravel upgrade rules

Closed this issue · 6 comments

hirenkeraliya commented

I check all the rules and upgrade for the laravel 7 to 8 and 8 to 9.
I got the rules can we make a set for this laravel upgrade?
if you agree then I will do that.

samsonasik commented

The set seems exists until laravel 8

https://github.com/rectorphp/rector-laravel/blob/6bee4284d50e84eb5d027ff266801b2a1c903a33/src/Set/LaravelLevelSetList.php#L64

feel free to create new set list PR until laravel 9

TomasVotruba commented

Hi, that would be very useful!

Please go for it 👍

hirenkeraliya commented

@samsonasik oh it's great. do we have documentation for it anywhere? if not then we also need to do this first.

samsonasik commented

@hirenkeraliya there is documentation for use set https://github.com/rectorphp/rector-laravel#use-sets , feel free to create documentation improvement PR.

hirenkeraliya commented

@samsonasik

Initially, I added a new rector class at that time I did it using the package development blog post.

But, I tried it today it's not working. And, also the rector configuration is changed. So, how can we debug when we want to print the object like $methodCall?

Can you able to help me with this?

  • 👍1
samsonasik commented

You can debug with print_node($node) or dump_node($node),