-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
bug: Fix PhpdocVarWithoutNameFixer with Closure with $this #6979
bug: Fix PhpdocVarWithoutNameFixer with Closure with $this #6979
Conversation
Sorry @mvorisek I did not have time during the weekend to review this, so it wasn't included in 3.17 release... Will look as soon as possible, but this week and upcoming I have pretty busy (PHPers conference 😁). |
Co-authored-by: Greg Korba <wirone@gmail.com>
Co-authored-by: Greg Korba <wirone@gmail.com>
4635eef
to
7206caa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Technically these changes are fine, I wonder only if $this
is that special. Could the type be self
instead or that would be differently "understood" by some tools?
@kubawerlos |
Thanks @mvorisek 🍻 |
…xer#6979) Co-authored-by: Greg Korba <greg@codito.dev>
fix #6959