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

add version 9 to avoid the check with version 8 #3544

Closed
wants to merge 2 commits into from
Closed

add version 9 to avoid the check with version 8 #3544

wants to merge 2 commits into from

Conversation

michael-wirth
Copy link

No description provided.

@michael-wirth
Copy link
Author

#3510 rebased as requested

@jcchavezs jcchavezs requested a review from shakuzen June 22, 2023 09:19
@shakuzen
Copy link
Member

Thank you for the pull request, but this still doesn't test that our code works with Elasticsearch 8, as I requested be added in #3510 (review).

@fazilali
Copy link
Contributor

can we get this merged?

@shakuzen
Copy link
Member

shakuzen commented Jul 6, 2023

can we get this merged?

Not until my review is addressed.

@fazilali
Copy link
Contributor

fazilali commented Jul 26, 2023

@shakuzen I have created another PR by addressing your review comment. can you take a look. #3552

@shakuzen
Copy link
Member

Closing in favor of #3552

@shakuzen shakuzen closed this Jul 27, 2023
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.

3 participants