Skip to content

Conversation

@skonto
Copy link
Contributor

@skonto skonto commented Aug 16, 2016

What changes were proposed in this pull request?

How was this patch tested?

Doc no test needed.

@SparkQA
Copy link

SparkQA commented Aug 16, 2016

Test build #63851 has finished for PR 14667 at commit ab82325.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Aug 16, 2016

Test build #63852 has finished for PR 14667 at commit 46d9962.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

Copy link
Member

Choose a reason for hiding this comment

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

OK by me, even in the UK English spelling.

Copy link
Contributor Author

@skonto skonto Aug 17, 2016

Choose a reason for hiding this comment

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

@srowen honour vs honor you mean? Missed that...

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, probably not worth worrying about unless you have a spare minute... I suppose the docs consistently use US spelling but it's hardly a big deal.

@skonto
Copy link
Contributor Author

skonto commented Aug 17, 2016

@srowen fixed

@SparkQA
Copy link

SparkQA commented Aug 17, 2016

Test build #63926 has finished for PR 14667 at commit 6d4f046.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@srowen
Copy link
Member

srowen commented Aug 18, 2016

Merged to master

@asfgit asfgit closed this in b81421a Aug 18, 2016
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