elastic / eland

Python Client and Toolkit for DataFrames, Big Data, Machine Learning and ETL in Elasticsearch
https://eland.readthedocs.io
Apache License 2.0
18 stars 98 forks source link

[7.x] Format code for linting failures #562

Closed davidkyle closed 1 year ago

davidkyle commented 1 year ago

Also pins the Elasticsearch client <8 for the 7.x branch

Follow up to #560

picandocodigo commented 1 year ago

@davidkyle I've just enabled Buildkite builds for pull requests from forks since this wasn't building. But I think the PR might need to be updated for Buildkite to pick up the change and trigger the build.

davidkyle commented 1 year ago

I've just enabled Buildkite builds for pull requests from forks

Ah I guess that is why the CI on https://github.com/elastic/eland/pull/559 hung. Thanks @picandocodigo

picandocodigo commented 1 year ago

Something is wrong on the configuration side, since after enabling the build on forks, it's still not working and the option goes back to unchecked. I'll look into it and report back once I'm certain it's working as intended.