Skip to content

[CodingStyle] Remove next node attribute on NewlineAfterStatementRector#3525

Merged
TomasVotruba merged 11 commits intomainfrom
remove-next-node-newline
Mar 27, 2023
Merged

[CodingStyle] Remove next node attribute on NewlineAfterStatementRector#3525
TomasVotruba merged 11 commits intomainfrom
remove-next-node-newline

Conversation

@samsonasik
Copy link
Copy Markdown
Member

@samsonasik samsonasik commented Mar 26, 2023

This remove AttributeKey::NEXT_NODE usage on NewlineAfterStatementRector,

Benefit

Limitation

  • this no longer work on spacing new line for multiple namespaces,eg:
namespace A{}
namespace B{}

as there is no parent of namespace to loop, see https://github.com/rectorphp/rector-src/pull/3525/files#diff-6169f918307ebebbb7127ccd6c653513c3269acb124e30add820d370fe3a0eac

Ref rectorphp/rector#7849

@samsonasik samsonasik marked this pull request as draft March 26, 2023 18:52
@samsonasik samsonasik force-pushed the remove-next-node-newline branch from 17533e4 to 97e96d5 Compare March 26, 2023 19:22
@samsonasik samsonasik marked this pull request as ready for review March 26, 2023 19:27
@samsonasik
Copy link
Copy Markdown
Member Author

All checks have passed 🎉 @TomasVotruba it is ready for review.

@TomasVotruba TomasVotruba merged commit ae2355d into main Mar 27, 2023
@TomasVotruba TomasVotruba deleted the remove-next-node-newline branch March 27, 2023 07:28
@TomasVotruba
Copy link
Copy Markdown
Member

Thank you 👏 👏 👏

samsonasik added a commit that referenced this pull request May 8, 2023
…or (#3525)

Co-authored-by: GitHub Action <actions@github.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

Development

Successfully merging this pull request may close these issues.

3 participants