Skip to content

Use ConfigureAwait(false)#74982

Merged
MichalStrehovsky merged 1 commit intodotnet:mainfrom
MichalStrehovsky:configureawait
Sep 5, 2022
Merged

Use ConfigureAwait(false)#74982
MichalStrehovsky merged 1 commit intodotnet:mainfrom
MichalStrehovsky:configureawait

Conversation

@MichalStrehovsky
Copy link
Member

@MichalStrehovsky MichalStrehovsky commented Sep 2, 2022

#74825 (review)

I don't think we need to run on the same thread.

Cc @tlakollo

dotnet#74825 (review)

I don't think we need to run on the same thread.
@ghost ghost assigned MichalStrehovsky Sep 2, 2022
@MichalStrehovsky MichalStrehovsky changed the title Use ConfigureAwait(true) Use ConfigureAwait(false) Sep 2, 2022
Copy link
Member

@stephentoub stephentoub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you.

@MichalStrehovsky MichalStrehovsky merged commit 4940059 into dotnet:main Sep 5, 2022
@MichalStrehovsky MichalStrehovsky deleted the configureawait branch September 5, 2022 00:53
@MichalStrehovsky
Copy link
Member Author

OSX failure is #75005.

@ghost ghost locked as resolved and limited conversation to collaborators Oct 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants