Skip to content

Conversation

@AlekseyTs
Copy link
Contributor

Related to #41704.

@AlekseyTs AlekseyTs requested a review from a team as a code owner March 16, 2020 23:13
@AlekseyTs
Copy link
Contributor Author

@cston, @RikkiGibson, @dotnet/roslyn-compiler Please review

@AlekseyTs
Copy link
Contributor Author

@RikkiGibson, @dotnet/roslyn-compiler Please review, need a second sign-off for a small change.

return node.DescendantNodesAndSelf(child => !IsNestedFunction(child)).
OfType<AwaitExpressionSyntax>().Any(); // PROTOTYPE(SimplePrograms): Recognize other async operations.
return node.DescendantNodesAndSelf(child => !IsNestedFunction(child)).Any(node =>
{
Copy link
Member

Choose a reason for hiding this comment

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

nit: the indentation is really unfortunate for reviewing on github.

Copy link
Member

@jcouv jcouv 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 (iteration 1)

@jcouv jcouv self-assigned this Mar 17, 2020
@AlekseyTs AlekseyTs merged commit c7d7f9b into dotnet:features/SimplePrograms Mar 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants