Skip to content

Conversation

BillWagner
Copy link
Member

Fixes #16634

This may be easier to review by individual commits:

  1. Add that IAsyncEnumerable<T> is a valid return type for an async method. Add a new project and sample code for IAsyncEnumerable.
  2. Fix style issues and remove unused anchors. (Those anchors were added during migration).
  3. Moves the old snippet files without any changes.
  4. Adapts the code to use the (new) preferred syntax. Update code to compile in a single project.

Fixes dotnet#16634

This commit: Add that `IAsyncEnumerable<T>` is a valid return type for an async method. Add a new project and sample code for IAsyncEnumerable.
In this commit, fix style issues and remove unused anchors. (Those anchors were added during migration).
This commit moves the old snippet files without any changes.
The existing sample code was in the /samples folder. This commit moves those into the ./snippets folder, and adapts the includes to use the (new) preferred syntax.
Copy link
Contributor

@tdykstra tdykstra left a comment

Choose a reason for hiding this comment

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

Looks good, and the code all runs perfectly.

BillWagner and others added 2 commits April 14, 2020 17:30
Co-Authored-By: Tom Dykstra <tdykstra@microsoft.com>
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.

Missing C# 8 documentation

3 participants