-
Notifications
You must be signed in to change notification settings - Fork 534
allow contravariant use of Subscribers in Publisher.subscribe #104
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
Comments
+1 |
2 similar comments
👍 |
👍 |
PR opened; @purplefox @normanmaurer @mariusaeriksen @tmontgomery please vote, perhaps we can get this into M2 which should then hopefully have the final interfaces for 1.0. |
+1 |
4 votes == ok ? |
still lacking one of Red Hat / Twitter / Kaazing according to the rules |
@rkuhn was not really able to follow the work much lately. But from what I read here it makes sense. So +1 I would say. |
Coolio, so I guess that this can go into 0.4.0.M2 today. @smaldini can you do the honors? (i.e. press the merge button) |
👍 |
Cheers everyone ! |
Since Subscriber is inherently contravariant in its type parameter, it would be appropriate to express that at the only use-site we define:
The text was updated successfully, but these errors were encountered: