Skip to content

Renaming ProtocolConnetion to ProtocolConnection#265

Merged
dbaeumer merged 1 commit intomicrosoft:masterfrom
object88:master
Oct 16, 2017
Merged

Renaming ProtocolConnetion to ProtocolConnection#265
dbaeumer merged 1 commit intomicrosoft:masterfrom
object88:master

Conversation

@object88
Copy link
Contributor

Reading through the code, I found a typo in one of the object names. It's unclear to me how far outside of this project this might extend, however. Please let me know if there is any more work around this.

@msftclas
Copy link

@object88,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.

It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@msftclas
Copy link

msftclas commented Oct 15, 2017

CLA assistant check
All CLA requirements met.

@msftclas
Copy link

@object88, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.

Thanks, Microsoft Pull Request Bot

@dbaeumer
Copy link
Member

Thanks. To make this non breaking and will add a type def for the old name.

@dbaeumer dbaeumer merged commit c5d57c1 into microsoft:master Oct 16, 2017
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.

3 participants