Skip to content

Conversation

@sharpist
Copy link

@sharpist sharpist commented Jul 9, 2018

I renamed the variable "infoTask" to "integerTask".

I renamed the variable "infoTask" to "integerTask".
@sharpist sharpist requested a review from BillWagner as a code owner July 9, 2018 22:23
@BillWagner
Copy link
Member

BillWagner commented Jul 12, 2018

@sharpist This is a good change.

Before I approve it and merge it, can you create a corresponding PR to update the variable in the article where this sample appears. It's here: https://github.com/dotnet/docs/blob/master/docs/csharp/programming-guide/concepts/async/async-return-types.md

Thanks for contributing to docs!

[UPDATE] It looks like @rpetrusha made the change in the article in PR dotnet/docs#6414

I'll :shipit: now.

@BillWagner BillWagner added the ✨ 1st-time samples contributor! Indicates PRs from new contributors to the samples repository label Jul 12, 2018
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 making this change @sharpist

I'll :shipit: now.

@BillWagner BillWagner merged commit 1969c72 into dotnet:master Jul 12, 2018
@mairaw mairaw added the 📁 Repo - samples Indicates PRs done in the samples repo. label Jul 12, 2018
@sharpist
Copy link
Author

Hi! @BillWagner, @mairaw, thank you for your attention!

karelz pushed a commit to karelz/samples that referenced this pull request Aug 31, 2018
I renamed the variable "infoTask" to "integerTask".
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. ✨ 1st-time samples contributor! Indicates PRs from new contributors to the samples repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants