Skip to content
This repository was archived by the owner on Aug 2, 2022. It is now read-only.

Conversation

abbashus
Copy link
Contributor

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@dai-chen
Copy link
Member

Thanks for the changes! Could you post a sample for what does this look like after merge?

@abbashus
Copy link
Contributor Author

Looks like this. Wanted to add codecov as well, but requires little more setup. Will add in next PR.

Screen Shot 2020-05-26 at 10 11 01 AM

@dai-chen
Copy link
Member

Looks like this. Wanted to add codecov as well, but requires little more setup. Will add in next PR.

Screen Shot 2020-05-26 at 10 11 01 AM

Cool. Thanks!

@abbashus abbashus merged commit 210c8fe into opendistro-for-elasticsearch:master May 26, 2020
penghuo pushed a commit to penghuo/sql that referenced this pull request Jun 17, 2020
penghuo pushed a commit that referenced this pull request Jun 17, 2020
(cherry picked from commit 210c8fe)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants