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.

Sign up for GitHub

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

Jump to bottom

Tiny typo to Painless dispatch justification #24588

Merged
merged 1 commit into from
May 11, 2017
Merged

Tiny typo to Painless dispatch justification #24588

merged 1 commit into from
May 11, 2017

Conversation

chrisivens
Copy link
Contributor

No description provided.

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@nik9000 nik9000 added :Plugin Lang Painless >docs General docs changes labels May 10, 2017
@nik9000
Copy link
Member

nik9000 commented May 10, 2017

Thanks @chrisivens! This is correct. Can you sign the CLA so I can merge it? We need the CLA for all changes regardless of their size.

@chrisivens
Copy link
Contributor Author

I signed it shortly before the PR. Let me know if you can't find it.

@nik9000
Copy link
Member

nik9000 commented May 11, 2017

I signed it shortly before the PR. Let me know if you can't find it.

Sometimes things take some time to go through the pipe. Commenting on the issue causes it to recheck so that did the trick.

@nik9000 nik9000 merged commit d447b79 into elastic:master May 11, 2017
nik9000 pushed a commit that referenced this pull request May 11, 2017
@chrisivens @nik9000
Docs: Tiny typo to Painless dispatch justification (#24588)
5ff0824
nik9000 pushed a commit that referenced this pull request May 11, 2017
@chrisivens @nik9000
Docs: Tiny typo to Painless dispatch justification (#24588)
656a14a
@nik9000
Copy link
Member

nik9000 commented May 11, 2017

Thanks @chrisivens! I've merged to master and cherry-picked to 5.x and 5.4 so it should be live in the docs in a half hour or so!

@clintongormley clintongormley added :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache and removed :Plugin Lang Painless labels Feb 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
:Core/Infra/Scripting Scripting abstractions, Painless, and Mustache >docs General docs changes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@chrisivens @elasticmachine @nik9000 @clintongormley