Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Use nameof in System.Collections.*#7296

Merged
stephentoub merged 1 commit intodotnet:masterfrom
eadordzhiev:nameof-collections
Mar 29, 2016
Merged

Use nameof in System.Collections.*#7296
stephentoub merged 1 commit intodotnet:masterfrom
eadordzhiev:nameof-collections

Conversation

@eadordzhiev
Copy link
Contributor

Replaces the hardcoded strings in System.Collections.* with nameof.
Contributes to #7290

/cc @stephentoub @jamesqo

@dnfclas
Copy link

dnfclas commented Mar 28, 2016

Hi @eadordzhiev, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

@dnfclas
Copy link

dnfclas commented Mar 28, 2016

@eadordzhiev, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.

Thanks, DNFBOT;

@eadordzhiev
Copy link
Contributor Author

@dotnet-bot Test this please (something's wrong with the CI)

@stephentoub
Copy link
Member

LGTM

@jamesqo
Copy link
Contributor

jamesqo commented Mar 28, 2016

LGTM as well. 👍

@stephentoub
Copy link
Member

Test Innerloop OSX Debug Build and Test please

@stephentoub stephentoub merged commit 59d66dc into dotnet:master Mar 29, 2016
@eadordzhiev eadordzhiev deleted the nameof-collections branch March 29, 2016 07:38
@karelz karelz modified the milestone: 1.0.0-rtm Dec 3, 2016
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
…ions

Use nameof in System.Collections.*

Commit migrated from dotnet/corefx@59d66dc
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants