Skip to content

Return ValueTask in loggers and interceptors (merge to preview6) #21158

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jun 6, 2020

Conversation

roji
Copy link
Member

@roji roji commented Jun 6, 2020

Closes #21109

(cherry picked from commit 9f316e5)

@roji roji requested review from ajcvickers and AndriySvyryd June 6, 2020 09:35
@roji roji force-pushed the ValueTaskStuffPreview6 branch from e44ce0c to 48f3479 Compare June 6, 2020 11:51
@roji roji merged commit 869b4af into release/5.0-preview6 Jun 6, 2020
@roji roji deleted the ValueTaskStuffPreview6 branch June 6, 2020 17:47
wtgodbe pushed a commit that referenced this pull request Jun 8, 2020
* [release/5.0-preview6] Update dependencies from dotnet/arcade (#21111)

* Update dependencies from https://github.com/dotnet/arcade build 20200530.1

Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Helix.Sdk
 From Version 5.0.0-beta.20261.9 -> To Version 5.0.0-beta.20280.1

* [master] Update dependencies from dotnet/arcade (#21091)

* Update dependencies from https://github.com/dotnet/arcade build 20200528.4

Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Helix.Sdk
 From Version 5.0.0-beta.20261.9 -> To Version 5.0.0-beta.20278.4

* Update dotnet on helix

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Brennan <brecon@microsoft.com>

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: dotnet-maestro[bot] <42748379+dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Brennan <brecon@microsoft.com>

* Return ValueTask in loggers and interceptors (#21152) (#21158)

Closes #21109

(cherry picked from commit 9f316e5)

Co-authored-by: dotnet-maestro[bot] <42748379+dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Brennan <brecon@microsoft.com>
Co-authored-by: Shay Rojansky <roji@roji.org>
wtgodbe pushed a commit that referenced this pull request Jun 9, 2020
* [release/5.0-preview6] Update dependencies from dotnet/arcade (#21111)

* Update dependencies from https://github.com/dotnet/arcade build 20200530.1

Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Helix.Sdk
 From Version 5.0.0-beta.20261.9 -> To Version 5.0.0-beta.20280.1

* [master] Update dependencies from dotnet/arcade (#21091)

* Update dependencies from https://github.com/dotnet/arcade build 20200528.4

Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Helix.Sdk
 From Version 5.0.0-beta.20261.9 -> To Version 5.0.0-beta.20278.4

* Update dotnet on helix

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Brennan <brecon@microsoft.com>

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: dotnet-maestro[bot] <42748379+dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Brennan <brecon@microsoft.com>

* Return ValueTask in loggers and interceptors (#21152) (#21158)

Closes #21109

(cherry picked from commit 9f316e5)

* Add ConfigureAwait(false) (#21110) (#21185)

Set up Microsoft.CodeAnalysis.FxCopAnalyzers with only the
ConfigureAwait rule enabled.

Closes #10164

(cherry picked from commit e1c9a3a)

Co-authored-by: dotnet-maestro[bot] <42748379+dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Brennan <brecon@microsoft.com>
Co-authored-by: Shay Rojansky <roji@roji.org>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants