Skip to content

[Performance] [PostRector] Only check phpdocinfo on Stmt and Param on NameImportingPostRector#4558

Merged
samsonasik merged 3 commits intomainfrom
samsonasik-patch-1
Jul 20, 2023
Merged

[Performance] [PostRector] Only check phpdocinfo on Stmt and Param on NameImportingPostRector#4558
samsonasik merged 3 commits intomainfrom
samsonasik-patch-1

Conversation

@samsonasik
Copy link
Copy Markdown
Member

@TomasVotruba @staabm this is to avoid unnecessary check phpdocinfo on Expr or Identifier.

Ref rectorphp/rector#8077

…PostRector

@TomasVotruba @staabm this is to avoid unnecessary check phpdocinfo on Expr or Identifier.

Ref rectorphp/rector#8077
@samsonasik samsonasik requested a review from TomasVotruba as a code owner July 20, 2023 20:59
@samsonasik samsonasik changed the title [PostRector] Only check phpdocinfo on Stmt and Param on NameImportingPostRector [Performance] [PostRector] Only check phpdocinfo on Stmt and Param on NameImportingPostRector Jul 20, 2023
@samsonasik
Copy link
Copy Markdown
Member Author

All checks have passed 🎉 @TomasVotruba I am merging it ;)

@samsonasik samsonasik merged commit f403ca0 into main Jul 20, 2023
@samsonasik samsonasik deleted the samsonasik-patch-1 branch July 20, 2023 21:05
@TomasVotruba
Copy link
Copy Markdown
Member

Thank you 🤗

@staabm
Copy link
Copy Markdown
Contributor

staabm commented Jul 21, 2023

thanks for working on it. I just tried the change locally and it did not yield any improvements.

will investigate further

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.

4 participants