-
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
Rename all operators that are duplicates of static operators to xWith
variants
#3927
Comments
The plan here, of course, is to add aliases and deprecations in the short term. |
Go for it, one thing less that can cause confusion. While at this, can you make it so that (Not sure this is possible at all) the old operators can't be found anymore by the auto-import tools in editors? |
I ran into this over the weekend, and spent several hours trying to work out why |
As someone that spends a fair bit of time introducing people to rxjs, this is an early pain point. While the two import locations make sense once you're experienced, it's something that seems really arbitrary early on, and the type errors you get from it are not helpful or enlightening to beginners. And even for experienced users it degrades the editor experience... Would be awesome to see fixed in v7! |
I think renaming them is a good idea because this confusion keeps popping on StackOverflow regularly and from the error it throws it's not obvious what you're doing wrong. |
I'm now thinking if when some static operators like |
@benlesh |
Following things should be considered:
|
Closing this because it has been done - except for |
concat -> concatWith
zip -> zipWith
merge -> mergeWith
etc.
Related #3798
The text was updated successfully, but these errors were encountered: