-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Expose SocketAsyncEventArgs constructor for suppressing the execution context #706
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thanks.
/azp run |
Azure Pipelines successfully started running 10 pipeline(s). |
src/libraries/System.Net.Sockets/src/System/Net/Sockets/SocketAsyncEventArgs.cs
Outdated
Show resolved
Hide resolved
…AsyncEventArgs.cs Co-Authored-By: Stephen Toub <stoub@microsoft.com>
/azp run |
Azure Pipelines successfully started running 10 pipeline(s). |
/azp run |
Commenter does not have sufficient privileges for PR 706 in repo dotnet/runtime |
@ViktorHofer @safern is it ok? |
never mind #718 |
/azp run runtime-libraries outerloop |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-libraries |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-libraries |
Commenter does not have sufficient privileges for PR 706 in repo dotnet/runtime |
@davidsh I joined external-ci-access group but seem that it's not retroactive, so for now I cannot re-run ci by myself here. |
/azp run runtime-libraries |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-libraries outerloop |
Azure Pipelines successfully started running 1 pipeline(s). |
closes https://github.com/dotnet/corefx/issues/32582
cc: @scalablecory @stephentoub