-
-
Notifications
You must be signed in to change notification settings - Fork 35
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
[PHPStan Baseline] Fix a few errors #216
Conversation
* @return self<NewTKey, NewT> | ||
* @return self<mixed, mixed> |
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.
these template types cannot be inferred from anywhere, so the better option here is to just use mixed
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.
Fair enough!
@@ -212,6 +212,7 @@ | |||
* @template-extends Prependable<TKey, T> | |||
* @template-extends Productable<TKey, T> | |||
* @template-extends Randomable<TKey, T> | |||
* @template-extends Rangeable<TKey, T> |
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.
somehow this was missing 😄
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.
Nice catch :-)
@@ -9,6 +9,7 @@ | |||
|
|||
namespace loophp\collection\Contract\Operation; | |||
|
|||
use Closure; |
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.
found a couple of these where the import was missing but the Closure
was mentioned in the doc blocks. Because of the missing import PHPStan couldn't resolve the type
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.
Indeed! Nice ones!
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.
👍
@@ -9,6 +9,7 @@ | |||
|
|||
namespace loophp\collection\Contract\Operation; | |||
|
|||
use Closure; |
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.
Indeed! Nice ones!
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.
Approved!
Thanks 😁 . I'll merge this once the checks have finished |
Thanks !! |
This PR:
Follows #212