ci: Consolidate requirements into pyproject.toml#7565
ci: Consolidate requirements into pyproject.toml#7565CaedenPH wants to merge 26 commits intoTheAlgorithms:masterfrom
pyproject.toml#7565Conversation
|
@cclauss Never used pyproject.toml, so expecting lots of changes |
|
We could also move our codespell, mypy, and pyupgrade args from .pre-commit into pyproject.toml. Please run https://pypi.org/project/validate-pyproject to validate the file and let's add that tool to pre-commit. |
How would this be done? |
Co-authored-by: Christian Clauss <cclauss@me.com>
|
The only benefit we get in using Build system is used to provide a backend like poetry, flit or setuptools to actually build the Python package (wheels). For our use case, |
|
Let's use Would we be able to use Could |
|
As I said, this won't work as |
|
I fail to understand the logic. |
dhruvmanila
left a comment
There was a problem hiding this comment.
I guess I must've misunderstood something, if it works then it's fine @cclauss
Co-authored-by: Dhruv Manilawala <dhruvmanila@gmail.com>
| "requests", | ||
| "rich", | ||
| "scikit-fuzzy", | ||
| "sklearn", |
There was a problem hiding this comment.
| "sklearn", | |
| "scikit-learn", |
| "sklearn", | ||
| "statsmodels", | ||
| "sympy", | ||
| "tensorflow", |
There was a problem hiding this comment.
| "tensorflow", | |
| # "tensorflow", # Not yet compatible with Python 3.11 |
Describe your change:
Consolidate the requirements into
pyproject.tomlhttps://pip.pypa.io/en/stable/reference/build-system/pyproject-toml/
Checklist:
Fixes: #{$ISSUE_NO}.