-
Notifications
You must be signed in to change notification settings - Fork 15
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
some deprecations #79
Comments
Also remove an unused import. Refs #79
With the addition of UriInterface as a return type hint, Phan is now complaining:
|
Yeah, I just ran into phpstan complaining in another project. I'm never quite sure when to return self, static or the class/interface name itself. |
It's a bit strange. It's like Phan is not loading vendor classes, but it must be because surely otherwise it'd be complaining about a bunch of other stuff. |
Oh, Phan was not loading the vendor directory! Fixing now. |
I saw these. I think it's okay to simply follow the suggestions, but can't remember exactly if they'd work for PHP 7.2.
The text was updated successfully, but these errors were encountered: