-
-
Notifications
You must be signed in to change notification settings - Fork 849
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
PHP Inspect Extended fix #1572
PHP Inspect Extended fix #1572
Conversation
436f4f8
to
edd0369
Compare
edd0369
to
da52ae7
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.
From now on we need to prefix functions by \
? Good to know!
@jewome62 yes I know the whys and what (though thx for the link it's interesting - /e I starred it a while ago haha). Just that we already discussed it last year IIRC and we didn't merge it. Good to know we're moving forward that's all :). |
Only some functions can be optimized. A PHP CS Fixer rule to replace only those ones is planned IIRC. |
Thanks @jewome62 ! |
PHP Inspect Extended fix
Fix some suggestion from PHP Inpector Extended into PHPStorm