-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
Add return types - batch 1/n #42504
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
Add return types - batch 1/n #42504
Conversation
d07ae36
to
81bf1f8
Compare
📗 (psalm failure are false-positives here.) |
All reactions
Sorry, something went wrong.
*/ | ||
public static function create(string $file, CacheItemPoolInterface $fallbackPool) | ||
public static function create(string $file, CacheItemPoolInterface $fallbackPool): CacheItemPoolInterface |
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.
public static function create(string $file, CacheItemPoolInterface $fallbackPool): CacheItemPoolInterface | |
public static function create(string $file, CacheItemPoolInterface $fallbackPool): static |
I don't quite understand why the previously annotated return type was so wide, tbh.
Sorry, something went wrong.
All reactions
81bf1f8
to
1564887
Compare
derrabus
dunglas
jderusse
lyrixx
xabbuh
yceruto
Successfully merging this pull request may close these issues.
Extracted from #42496 for components that shouldn't be extended very often.