Skip to content
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 Observable.Amb to Race #308

Closed
neuecc opened this issue Feb 13, 2025 · 1 comment
Closed

Rename Observable.Amb to Race #308

neuecc opened this issue Feb 13, 2025 · 1 comment

Comments

@neuecc
Copy link
Member

neuecc commented Feb 13, 2025

Amb is a name that has been continuously used since the original Rx, with its etymology coming from LISP's 'amb', but it's not an intuitive name for its behavior.
I think it would be more appropriate to change it to something like Race.
It was my mistake not to change the name during R3's initial release.
For the transition, I suggest adding Race while marking Amb as [Obsolete].
What do you think?

@neuecc
Copy link
Member Author

neuecc commented Feb 15, 2025

v.1.3.0 changed.

@neuecc neuecc closed this as completed Feb 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant