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

Enable recently added Microsoft.CodeAnalysis.NetAnalyzers rules #53158

Merged
merged 1 commit into from
May 28, 2021

Conversation

stephentoub
Copy link
Member

No description provided.

@stephentoub stephentoub added the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label May 24, 2021
@stephentoub stephentoub force-pushed the enablenewrules branch 2 times, most recently from ed5c7f5 to 6778073 Compare May 24, 2021 14:07
@stephentoub stephentoub requested a review from marek-safar as a code owner May 24, 2021 14:07
@stephentoub stephentoub force-pushed the enablenewrules branch 2 times, most recently from 77c814b to de9124c Compare May 24, 2021 22:09
@stephentoub stephentoub removed the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label May 25, 2021
@stephentoub stephentoub requested a review from jeffhandley May 26, 2021 00:53
@stephentoub stephentoub added this to the 6.0.0 milestone May 26, 2021
@@ -454,21 +454,14 @@ public override long Position
set => throw new NotSupportedException();
}

public override async Task<int> ReadAsync(byte[] buffer, int offset, int count, CancellationToken cancellationToken)
public override Task<int> ReadAsync(byte[] buffer, int offset, int count, CancellationToken cancellationToken)
Copy link
Member Author

Choose a reason for hiding this comment

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

@kjpou1, would you mind reviewing the changes to this file?

Copy link
Contributor

Choose a reason for hiding this comment

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

@stephentoub stephentoub requested a review from buyaa-n May 28, 2021 11:03
Copy link
Contributor

@buyaa-n buyaa-n left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@stephentoub stephentoub merged commit 72ff0df into dotnet:main May 28, 2021
@stephentoub stephentoub deleted the enablenewrules branch May 28, 2021 18:46
@ghost ghost locked as resolved and limited conversation to collaborators Jun 27, 2021
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