Skip to content

Conversation

@ianomad
Copy link
Contributor

@ianomad ianomad commented Apr 18, 2017

  • Have you signed the contributor license agreement?
  • Have you followed the contributor guidelines?
  • If submitting code, have you built your formula locally prior to submission with gradle check?
  • If submitting code, is your pull request against master? Unless there is a good reason otherwise, we prefer pull requests against master and will backport as needed.
  • If submitting code, have you checked that your submission is for an OS that we support?
  • If you are submitting this code for a class then read our policy for that.

@karmi
Copy link
Contributor

karmi commented Apr 18, 2017

Hi @ianomad, we have found your signature in our records, but it seems like you have signed with a different e-mail than the one used in yout Git commit. Can you please add both of these e-mails into your Github profile (they can be hidden), so we can match your e-mails to your Github profile?

@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?

@dakrone dakrone merged commit 688fa30 into elastic:master Apr 19, 2017
@dakrone
Copy link
Member

dakrone commented Apr 19, 2017

Merged, thanks @ianomad!

@dakrone dakrone added >docs General docs changes v6.0.0-alpha1 and removed Awaiting CLA labels Apr 19, 2017
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Apr 19, 2017
* master:
  Wait for cluster to become quiescent between REST tests (elastic#24148)
  Plugins: Remove leniency for missing plugins dir (elastic#24173)
  Changing some grammar in docs (elastic#24164)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>docs General docs changes v6.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants