-
Notifications
You must be signed in to change notification settings - Fork 3k
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
zip operator's type definition is incompatible with RxJS v4 #1208
Comments
@david-driscoll , as well as #1209 I assume this is related with your current effort of introducing n-number parameter signatures for some operators, is my understanding correct? |
Yep, most of the work is done on the RxJS4 side, just need to migrate it over to RxJS5 without interuptions. Once #1086 lands, I was planning on adding more for |
Based on one of my previous attempts at getting the typings working, I have the following (at the time generated file) that has pretty much all they typings we're looking for. It's a little outdated, but I'll be pulling from there to start. https://github.com/david-driscoll/RxJS-1/blob/all-types/src/operator-typings.ts |
#1086 is now checked in, those change can be used as reference to other operators as well. |
Closing this issue via #1292. |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
The text was updated successfully, but these errors were encountered: