-
Notifications
You must be signed in to change notification settings - Fork 7
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
Overloaded .subscribe inference #10
Comments
Which of the two subscribe signatures do you think is more important? The apply function if the Observable class has the |
Both are important :/ |
We don't have to solve it right away - I don't know what's best way to go about it (since my guess is it's gonna be problem in most overloaded methods)... but we can keep the issue open to recognize limitations |
Yeah, unfortunately the original RxScala api suffers from the same problems. We could create extra functions instead of overloads, but I don't want to strain to far from the original API. |
Yea, straining from api is my biggest worry. That would just bring gap to newcomers trying to use typed rx with scala.js... but so will different naming =/ |
There is what I believe same inference problem as in #6 with subscribe method, which is very unfortunace... Can we do do something about it ?
The text was updated successfully, but these errors were encountered: