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

add Async.Choice to FSharp.Core #8

Closed
cloudRoutine opened this issue Oct 12, 2016 · 3 comments
Closed

add Async.Choice to FSharp.Core #8

cloudRoutine opened this issue Oct 12, 2016 · 3 comments

Comments

@cloudRoutine
Copy link
Owner

cloudRoutine commented Oct 12, 2016

add Async.Choice to FSharp.Core [10575069]

Submitted by Steffen Forkmann on 11/6/2015 12:00:00 AM
[ 12 votes ]

Async.Choice is super useful (e.g. very important in Paket) but hard to implement. There are couple of implementations floating around like http://www.fssnip.net/dO but it's ahrd to decide which one is correct. We should add it to the core.

Response

** by fslang-admin on 1/23/2016 12:00:00 AM **

Approved and completed, though not yet in any specific public release of F#.
PR dotnet/fsharp#744 completed.
Don Syme, F# Language Evolution

@cloudRoutine
Copy link
Owner Author

Posted by Eirik George Tsarpalis on 11/6/2015 4:38:00 AM

Btw, the implementation of http://www.fssnip.net/dO is flawed. I would consider using http://www.fssnip.net/dN instead.

@cloudRoutine
Copy link
Owner Author

Posted by Steffen Forkmann on 11/6/2015 4:41:00 AM

If I remember correctly that version didn't work in paket since it doesn't cancel running http requests

@cloudRoutine
Copy link
Owner Author

Posted by Steffen Forkmann on 11/28/2015 4:48:00 AM

PR is at dotnet/fsharp#744

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant