Skip to content

F#6: migration from async{} to task{}#200

Merged
sergey-tihon merged 5 commits intomasterfrom
fsharp6
Aug 11, 2022
Merged

F#6: migration from async{} to task{}#200
sergey-tihon merged 5 commits intomasterfrom
fsharp6

Conversation

@sergey-tihon
Copy link
Member

No description provided.

@Thorium
Copy link
Member

Thorium commented Aug 7, 2022

I wouldn't recommend to use task until VS2022 17.3 is released with the fix for the F# compiler, handling task state machine properly. Meanwhile it may not affect this library directly, the users might be affected.

@Thorium
Copy link
Member

Thorium commented Aug 10, 2022

...which was released yesterday.

@sergey-tihon
Copy link
Member Author

@Thorium thank you, can you also please point me to bug that was fixed.

One of my tests fails because can't catch exception throw inside task{} ... can it be related?

@Thorium
Copy link
Member

Thorium commented Aug 10, 2022

I think it's not, rather async-exceptions are via Async.Catch where task is your traditional AggregateException.

@sergey-tihon sergey-tihon merged commit c0ce1ae into master Aug 11, 2022
@sergey-tihon sergey-tihon deleted the fsharp6 branch August 11, 2022 18:07
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

Successfully merging this pull request may close these issues.

2 participants