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 typo on Client API putTemplate, was putTempalte #26

Merged
merged 1 commit into from
Dec 8, 2016
Merged

fix typo on Client API putTemplate, was putTempalte #26

merged 1 commit into from
Dec 8, 2016

Conversation

gszr
Copy link
Contributor

@gszr gszr commented Dec 8, 2016

No description provided.

@DhavalKapil DhavalKapil merged commit 1218660 into DhavalKapil:master Dec 8, 2016
@DhavalKapil
Copy link
Owner

Thanks :)

@coveralls
Copy link

coveralls commented Dec 8, 2016

Coverage Status

Coverage remained the same at 91.264% when pulling 777ca12 on salazar:fix-typo into 9395f27 on DhavalKapil:master.

@pmusa
Copy link
Collaborator

pmusa commented Dec 8, 2016

Maybe we should add some test here :)

@DhavalKapil
Copy link
Owner

You mean test for every API function in Client.lua?

@pmusa
Copy link
Collaborator

pmusa commented Dec 8, 2016

Missing a typo like that is not good. How can we test it and make sure it does not happen?

@DhavalKapil
Copy link
Owner

Maybe comparing methods in src/elasticsearch/*.lua with the ones in https://github.com/elastic/elasticsearch/tree/master/rest-api-spec/src/main/resources/rest-api-spec/api ?

@pmusa
Copy link
Collaborator

pmusa commented Dec 8, 2016

Sounds like an option.

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.

None yet

4 participants