Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix some misspellings in code, comments and xml docs. #1230

Merged
merged 31 commits into from
Apr 4, 2016

Conversation

alexander-efremov
Copy link
Contributor

I've found and fixed some misspellings inside Octokit solution. There are a lot of changes, but they all very simple, one or two letter have been changed only. Hope this PR would be useful.

/cc @ryangribble, @shiftkey

@@ -41,7 +40,7 @@ public class TheCreateMethod
[IntegrationTest]
public async Task ForkCreatedForUserLoggedIn()
{
// The fork is created asynchronially by github and therefore it cannot
// The fork is created asynchronically by github and therefore it cannot
Copy link
Member

Choose a reason for hiding this comment

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

asynchronously

@@ -58,7 +57,7 @@ public class TheCreateMethod
[OrganizationTest]
public async Task ForkCreatedForOrganization()
{
// The fork is created asynchronially by github and therefore it cannot
// The fork is created asynchronically by github and therefore it cannot
Copy link
Member

Choose a reason for hiding this comment

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

asynchronously

@shiftkey
Copy link
Member

shiftkey commented Apr 3, 2016

Just a heads up: #1238 has a fix for the Travis issue - feel free to cherry-pick 8ec4bf3 so you can verify this PR is still correct

@alexander-efremov alexander-efremov deleted the fix-some-misspellings branch April 4, 2016 06:04
@alexander-efremov alexander-efremov restored the fix-some-misspellings branch April 4, 2016 06:04
@alexander-efremov
Copy link
Contributor Author

@shiftkey, all checks, pls, check this out whenever you want :)

@shiftkey
Copy link
Member

shiftkey commented Apr 4, 2016

@dampir thanks!

@shiftkey shiftkey merged commit 8a4abe2 into octokit:master Apr 4, 2016
@alexander-efremov alexander-efremov deleted the fix-some-misspellings branch April 5, 2016 06:05
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