Skip to content

Update helpers to use new multisearch types #2697

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 2 commits into from
Apr 4, 2025
Merged

Conversation

JoshMock
Copy link
Member

@JoshMock JoshMock commented Apr 4, 2025

  • Update helpers to use correct multisearch types
  • Stop copying project files to Dockerfile

JoshMock added 2 commits April 4, 2025 12:24
The spec combined definitions for search and multisearch bodies in
elastic/elasticsearch-specification#2960.
Slightly faster run times for codegen, hopefully.
@JoshMock
Copy link
Member Author

JoshMock commented Apr 4, 2025

CI failures are a chicken-and-egg problem that will be fixed on the next successful code generation, which is currently failing without the fix this PR makes. Will immediately rerun codegen after merging this.

@JoshMock JoshMock merged commit b2a4907 into main Apr 4, 2025
8 of 23 checks passed
@JoshMock JoshMock deleted the msearch-helper-fix branch April 4, 2025 17:32
github-actions bot pushed a commit that referenced this pull request Apr 4, 2025
* Update helpers to use correct multisearch types

The spec combined definitions for search and multisearch bodies in
elastic/elasticsearch-specification#2960.

* Stop copying project files to Dockerfile

Slightly faster run times for codegen, hopefully.

(cherry picked from commit b2a4907)
JoshMock added a commit that referenced this pull request Apr 4, 2025
* Update helpers to use correct multisearch types

The spec combined definitions for search and multisearch bodies in
elastic/elasticsearch-specification#2960.

* Stop copying project files to Dockerfile

Slightly faster run times for codegen, hopefully.

(cherry picked from commit b2a4907)

Co-authored-by: Josh Mock <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant