add ValueObservable.combineLatestN + tests#337
add ValueObservable.combineLatestN + tests#337lukepighetti wants to merge 3 commits intoReactiveX:masterfrom
Conversation
|
I actually really like the idea behind this change. It's something I'd considered in the past as well! One question about the implementation that I'd love some feedback on: It currently uses |
|
I was not aware that was a side-effect of using |
|
Heya @lukepighetti -- how are ya feeling about this change? |
|
If you can provide a quick and clear request for the change I'd be happy to implement it and push. My main concern is making sure this is consistent with your vision for |
|
One thing I was considering: I don't think most people want
Perhaps we could use that base for this change? |
Quick PR just to see what your appetite is for ValueObservable specific methods to preserve ValueObservable type