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

Typo 'Elastisearch' -> 'Elasticsearch' #23633

Merged
merged 2 commits into from
Mar 20, 2017
Merged

Conversation

maxg203
Copy link
Contributor

@maxg203 maxg203 commented Mar 17, 2017

Quick typo fix

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

@karmi
Copy link
Contributor

karmi commented Mar 17, 2017

Hi @maxg203, 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?

@qwerty4030
Copy link
Contributor

Here are some more instances of the same typo: https://github.com/elastic/elasticsearch/search?utf8=%E2%9C%93&q=Elastisearch

@maxg203
Copy link
Contributor Author

maxg203 commented Mar 19, 2017

@karmi That is a good point, I added that email as well. Everything should be good now.

@maxg203
Copy link
Contributor Author

maxg203 commented Mar 19, 2017

@qwerty4030 Thank you, I just updated this pull request to reflect those extra instances that you found.

@nik9000 nik9000 added :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts >docs General docs changes >test Issues or PRs that are addressing/adding tests v5.4.0 v6.0.0-alpha1 labels Mar 20, 2017
@nik9000 nik9000 merged commit faa6ade into elastic:master Mar 20, 2017
nik9000 pushed a commit that referenced this pull request Mar 20, 2017
Corrects `Elastisearch` to `Elasticsearch` in some docs and a packaging test utility.
@nik9000
Copy link
Member

nik9000 commented Mar 20, 2017

Thanks @maxg203 and @qwerty4030! I've merged and backported to 5.x.

jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Mar 22, 2017
* master:
  Fix typo in allocation explain API docs
  Add unit tests for ReverseNestedAggregator (elastic#23651)
  Revert "Revert "Build: Upgrade min gradle to 3.3 (elastic#23544)""
  Revert "Build: Upgrade min gradle to 3.3 (elastic#23544)"
  Build: Upgrade min gradle to 3.3 (elastic#23544)
  Fix took assertion in response filter test
  Search took time should use a relative clock
  Adds toString() to snapshot operations in progress
  Docs: fix a typo in transport client's put-mapping.asciidoc (elastic#23607)
  Use include-tagged macro for high level client docs (elastic#23438)
  Update fill-column in .dir-locals.el to 100 characters
  Setup keystore during integration tests (elastic#22966)
  Fix typo 'Elastisearch' -> 'Elasticsearch' (elastic#23633)
  Comment and blank line cleanups (elastic#23647)
  docs: guidelines for students and teachers (elastic#23648)
  Fix MapperService StackOverflowError (elastic#23605)
@mark-vieira mark-vieira added the Team:Delivery Meta label for Delivery team label Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts >docs General docs changes Team:Delivery Meta label for Delivery team >test Issues or PRs that are addressing/adding tests v5.4.0 v6.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants