Skip to content

Prep for 6.8 release #178

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

Merged
merged 8 commits into from
Mar 11, 2020
Merged

Prep for 6.8 release #178

merged 8 commits into from
Mar 11, 2020

Conversation

ezimuel
Copy link
Contributor

@ezimuel ezimuel commented Jan 7, 2020

This PR contains the API endpoints update for Elasticsearch 6.8.7. Moreover, it includes the unit test and YAML test for OSS and XPack.

This PR fixes also #179 exposing an xpack object. The XPack endpoints have been moved into the Direct client, there's no need anymore to specify XPack as plugin.

@sewi-cpan
Copy link
Contributor

f35efcb#diff-87ca88ecdb2370e98cca19dade0beee4 looks like you left over some debugging stuff.

The commit includes a tidy of many files which hides the actual logic changes. Tidy is nice and every publich source should be tidy, but don't try to backdoor-introduce it to a project.

@ezimuel
Copy link
Contributor Author

ezimuel commented Jan 14, 2020

@sewi-cpan thanks for reporting. I'm still working on this PR and I'll remove the data dump and other includes.

@ezimuel ezimuel changed the title Prep for 6.8.6 release [WIP: pPrep for 6.8.6 release Jan 14, 2020
@ezimuel ezimuel changed the title [WIP: pPrep for 6.8.6 release WIP: Prep for 6.8.6 release Jan 14, 2020
@ezimuel ezimuel added this to the 6.8 milestone Jan 14, 2020
@sewi-cpan
Copy link
Contributor

@sewi-cpan thanks for reporting. I'm still working on this PR and I'll remove the data dump and other includes.

Sorry, didn't recognize that you're maintainer :)
Some people tend to issue PRs on projects with minimal changes but huge formatting change.

@ezimuel
Copy link
Contributor Author

ezimuel commented Feb 14, 2020

@sewi-cpan don't worry. I'm glad that I have some code review. Thanks to you!

@ezimuel ezimuel changed the title WIP: Prep for 6.8.6 release WIP: Prep for 6.8 release Mar 10, 2020
@ezimuel ezimuel changed the title WIP: Prep for 6.8 release Prep for 6.8 release Mar 11, 2020
@ezimuel ezimuel merged commit 4f7a18e into elastic:master Mar 11, 2020
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.

2 participants