-
Notifications
You must be signed in to change notification settings - Fork 341
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 Future::select to Future::race #425
Comments
I like this name Future::race. It's familiar to me, than select. |
I'll update #405 with this change. |
While I agree that The reasoning is mainly that this requires the user to have a specific context (either being from the JS world or knowing the specifics of |
Related (here I talked about Good counterpoint to mine: rust-lang/futures-rs#760 (comment)
I believe we could mention "select" in the docs of |
This was closed by #405. |
Both JS and reactive streams use
race
for what we callselect
. So perhaps we should consider following suit.This is after feedback from @withoutboats that the name may not be right.
The text was updated successfully, but these errors were encountered: