-
-
Notifications
You must be signed in to change notification settings - Fork 92
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
vs IObservable<T>
#10
Comments
Hi I am totally new to the Subject - so what i'm saying is probably totally wrong - but as I am reading this artice: https://reactivex.io/intro.html At the bottom of the page it says:
So isn't the question here, if IObservable has these semantics? |
But another question I would also ask is what would be the simplest for the library? |
Btw if you rename this, you should definitely call it UnitedRx ;-) |
The reason for the change is described in the ReadMe. |
I think it would be better to create a different interface and write Reasons:
Possible problems:
|
This comment is a little off from the original issue raised, but I would like to see an extension method that can execute The purpose of converting to |
Hm, I suggested just a name, format. Of course we must support generic parametrization. And I meant, of course |
I strongly agree with that. I was only suggesting the addition of an API, not that the current API should be rearranged. I also believe that my proposal is a watershed. It comes down to this iuuse agenda in the end. If we choose not to stick with IObservable, we should present the importance of using IObservable. My proposal is based on the assumption that the significance of IObservable use is the limitation of Subscribe-only, and beyond that, I think it is better not to adopt it if it defines a different significance. |
The current API uses a custom definition that differs from
IObservable<T>
and is therefore incompatible as is (conversion is possible withAsIObservable
).Whether we should go with a custom definition or conform to
IObservable<T>
.The text was updated successfully, but these errors were encountered: