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.

Sign up for GitHub

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

Jump to bottom

Make use of PHPStan static reflection #3630

Merged
merged 2 commits into from
Jul 1, 2020
Merged

Make use of PHPStan static reflection #3630

merged 2 commits into from
Jul 1, 2020

Conversation

TomasVotruba
Copy link
Member

@TomasVotruba TomasVotruba commented Jul 1, 2020 •
edited
Loading

Closes #3490

  1. Use ReflectionProvider interface instead of Broker class in places where you're obtaining class/function/constant reflections. They have the same API.

There is no Broker in current Rector code 👍

@TomasVotruba TomasVotruba changed the title Make use of PHPStan static Make use of PHPStan static reflection Jul 1, 2020
@TomasVotruba TomasVotruba merged commit a922d77 into master Jul 1, 2020
@TomasVotruba TomasVotruba deleted the reflection-over-broker branch July 1, 2020 22:00
TomasVotruba added a commit that referenced this pull request Apr 19, 2023
@TomasVotruba
Updated Rector to commit e3e51469627756e976a8b8afb718d77caad3a0c4
203f525 
rectorphp/rector-src@e3e5146 [CodingStyle] Handle inner closure on FuncGetArgsToVariadicParamRector (#3630)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Taking advantage of static reflection
1 participant
@TomasVotruba