Skip to content

Conversation

@pkulikov
Copy link
Contributor

Supports dotnet/docs#5597


// <Snippet13>
public class List<T>
namespace ListExample
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added that namespace to be able to use the generic .NET List<T> in the foreach snippets.

A separate namespace per file might be a better fix.

Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

Thanks for adding this, and updating the corresponding text.

I'll :shipit: now, and then trigger a build on dotnet/docs#5597

Thanks again.

@BillWagner BillWagner merged commit c5b8f66 into dotnet:master May 25, 2018
@pkulikov pkulikov deleted the foreach-snippets branch May 25, 2018 13:53
@mairaw mairaw added the 📁 Repo - samples Indicates PRs done in the samples repo. label May 25, 2018
karelz pushed a commit to karelz/samples that referenced this pull request Aug 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📁 Repo - samples Indicates PRs done in the samples repo.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants