kedro-org / kedro

Kedro is a toolbox for production-ready data science. It uses software engineering best practices to help you create data engineering and data science pipelines that are reproducible, maintainable, and modular.
https://kedro.org
Apache License 2.0
9.49k stars 875 forks source link

Remove jmespath dependency #3797

Closed merelcht closed 3 months ago

merelcht commented 3 months ago

Description

Inspired by the discussion in #3659 on the number of dependencies on Kedro. It was used in the TemplatedConfigLoader https://github.com/kedro-org/kedro/blob/3529c29234ba86a9d3605e3a80d030518f06b311/kedro/config/templated_config.py and we forgot to remove it when TCL was deleted.

Development notes

All tests are passing, so I think we're good to go and remove it 🥳

Developer Certificate of Origin

We need all contributions to comply with the Developer Certificate of Origin (DCO). All commits must be signed off by including a Signed-off-by line in the commit message. See our wiki for guidance.

If your PR is blocked due to unsigned commits, then you must follow the instructions under "Rebase the branch" on the GitHub Checks page for your PR. This will retroactively add the sign-off to all unsigned commits and allow the DCO check to pass.

Checklist

noklam commented 3 months ago

https://github.com/fpgmaas/deptry

Maybe we can try something similar to detect unused dependencies automatically? This is for poetry project, maybe there is something for others.