Kaggle / kaggle-api

Official Kaggle API
Apache License 2.0
6.01k stars 1.06k forks source link

install-deps script is missing several dependencies #599

Open mgallifrey opened 2 days ago

mgallifrey commented 2 days ago

In preparing a fix for https://github.com/Kaggle/kaggle-api/issues/574, I noticed that hatch run install-deps did not install all of the necessary dependencies, causing hatch run compile to fail halfway through and leave all the generated code deleted.

It appears that in addition to the list yapf3 and python3-yapf packages, at least unzip, python3-toml, and java are also needed.

Happy to submit a PR for this.