Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Core] Always reset stopTraversal to false on next Rector visit #4182

Merged
merged 14 commits into from
Jun 11, 2023

Conversation

samsonasik
Copy link
Member

@samsonasik
Copy link
Member Author

@TomasVotruba by this, verify just reprinted node in RectifiedAnalyzer also no longer needed, as it always ensure traverse rector rule done before continue to other rector rule 66ba1ba

@samsonasik
Copy link
Member Author

Just reprinted node check seems still needed

@samsonasik
Copy link
Member Author

Phpdoc info update on each after refactor() seems no longer needed e6a4100

@samsonasik
Copy link
Member Author

All checks have passed 🎉 @TomasVotruba let's give it a try 👍

@samsonasik samsonasik merged commit bb609b2 into main Jun 11, 2023
42 checks passed
@samsonasik samsonasik deleted the fix-stop-traverser branch June 11, 2023 14:13
@samsonasik
Copy link
Member Author

@TomasVotruba it seems has 1 error, not syntax error, but ReflectionClass::getAttributes() usage

https://github.com/rectorphp/rector-src/actions/runs/5235899119/jobs/9453066468#step:24:11

on vendor/symfony/console/Command/Command.php, I will immediate revert with try another solution.

I think STOP_TRAVERSAL should not be in direct return on refactor() method.

@samsonasik
Copy link
Member Author

Reverting at #4183

samsonasik added a commit that referenced this pull request Jun 11, 2023
samsonasik added a commit that referenced this pull request Jun 11, 2023
* Remove PARENT_NODE from BetterStandardPrinter

* [ci-review] Rector Rectify

* Revert "[Core] Always reset stopTraversal to false on next Rector visit (#4182)" (#4183)

This reverts commit bb609b2.

---------

Co-authored-by: GitHub Action <actions@github.com>
Co-authored-by: Abdul Malik Ikhsan <samsonasik@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants