Skip to content

Conversation

@BillWagner
Copy link
Member

This sample code creates a list object, which is easily read as discussing the System.Collections.Generic.List class.

Contributes to dotnet/docs#7884

This sample code creates a list object, which is easily read as discussing the System.Collections.Generic.List class.

Contributes to dotnet/docs#7884
@BillWagner BillWagner requested a review from Thraka April 3, 2019 20:17
BillWagner added a commit to BillWagner/docs that referenced this pull request Apr 3, 2019
Fixes dotnet#7884

The `List<T>` example causes confusion with the .NET Standard type `System.Collections.Generic.List<T>`

Change the name of that type, update all references to the type name, and add a note to the .NET Standard type.

Depends on dotnet/samples#785
Copy link

@rpetrusha rpetrusha left a comment

Choose a reason for hiding this comment

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

LGTM, @BillWagner. You can merge when you're ready, and probably should rebuild dotnet/docs#11649.

@BillWagner BillWagner merged commit 7a218b2 into dotnet:master Apr 3, 2019
@BillWagner BillWagner deleted the change-confusing-generic-example branch April 3, 2019 20:35
BillWagner added a commit to dotnet/docs that referenced this pull request Apr 3, 2019
* change relative links

We prefer ~ - based links to samples, because those links are from another repository.

* Replace List with MyList

Fixes #7884

The `List<T>` example causes confusion with the .NET Standard type `System.Collections.Generic.List<T>`

Change the name of that type, update all references to the type name, and add a note to the .NET Standard type.

Depends on dotnet/samples#785

* Apply suggestions from code review

Co-Authored-By: BillWagner <wiwagn@microsoft.com>

* fix reference link.

* fix typo in link.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants