Skip to content
This repository has been archived by the owner on Mar 11, 2024. It is now read-only.

Use supported elasticsearch client #6

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

michaelkirk
Copy link

@michaelkirk michaelkirk commented Jun 15, 2023

This replaces the deprecated elasticsearch package with it's successor @elastic/elasticsearch.

This is required by pelias/schema#488

I did run the tests locally, but most of my confidence comes from it being used in my elastic8 fork of pelias/schema.

See the mostly* successful CI run https://github.com/michaelkirk-pelias/schema/actions/runs/5274562778/jobs/9539146582

*CI is only failing because I don't have docker privs set up - the unit and integration tests are passing on es7 and es8.

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.

1 participant