-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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
ESQL: Stop sending version in tests #108961
Conversation
Now that `version` is no longer required anywhere we can stop sending it in all of our tests. Closes elastic#108957
Pinging @elastic/es-analytical-engine (Team:Analytics) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Failures seem to be related to #108976 which I think we were masking. I'm going to merge if the last bwc looks good. |
💔 Backport failed
You can use sqren/backport to manually backport by running |
Fine. I'll do it with my fingers. |
Now that `version` is no longer required anywhere we can stop sending it in all of our tests. Closes elastic#108957
Backport: #108984 |
Now that
version
is no longer required anywhere we can stop sending it in all of our tests.Closes #108957