Conversation
|
I'd only merge this if the original was covariant as well: https://github.com/azjezz/psl/blob/next/src/Psl/Option/Option.php Otherwise it's not safe. |
|
It looks like it maybe shouldn't be covariant due to methods like There are cases that I'm unsure about how to handle, for example: I'm not sure about if it's a problem that should be handled with call-site variance (expecting I'm not sure at all about that last point, as I'm still understanding variance, but to me |
|
Hard to judge without a code sample on phpstan.org/try |
|
I'll close the pull request for now and open an issue with any specific example if I get an reproducer. |
No description provided.