search

elastic / elastic-transport-js

Transport classes and utilities shared among Node.js Elastic client libraries
Apache License 2.0
4 stars 24 forks source link

Upgrade Undici to v6.7.0 #85

Closed cosieLq closed 3 months ago

cosieLq commented 4 months ago

Closes #82

This PR tries to upgrade Undici to v6.7.0. I haven't run integration tests, and I'm curious if everything is still working well.

This is my first commit to this repo, and comments are very welcome! :relaxed:

cla-checker-service[bot] commented 4 months ago

💚 CLA has been signed

thomasconner commented 3 months ago

Any update on this?

JoshMock commented 3 months ago

Finally got a chance to pull this down and test it against the client. All unit and integration tests pass smoothly! Just need @cosieLq to sign that CLA and I can merge this for the next minor release.

JoshMock commented 3 months ago

@cosieLq Additionally, if you merge main into this branch, those Node 16 tests will be removed.

cosieLq commented 3 months ago

@cosieLq Additionally, if you merge main into this branch, those Node 16 tests will be removed.

@JoshMock I've signed the CLA and merged main to this branch. Thank you for your reviewing and testing!