Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: use pep621 pyproject.toml #372

Merged
merged 5 commits into from
Jul 20, 2022
Merged

build: use pep621 pyproject.toml #372

merged 5 commits into from
Jul 20, 2022

Conversation

tekumara
Copy link
Collaborator

@tekumara tekumara commented Jul 17, 2022

PEP 621 provides a standardised way for storing project metadata including dependencies declaratively in pyproject.toml, instead of setup.py which is very dynamic. PEP 621 is now supported by setuptools.

This PR uses project settings in pyproject.toml to configure setuptools to build the sdist and wheel.

The built distribution is the same, except the PKG-INFO now includes classifies, keywords, and the full text of the license.

NB: pyproject.toml dependencies are not yet supported by renovatebot (see #10187) or dependabot (see #3290)

Copy link
Contributor

@huynguyent huynguyent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we consider using poetry in this project in the future? We have been using it for a while at Seek and it's been great

@tekumara
Copy link
Collaborator Author

To pin down all the dependencies? Possibly, although it might make it less useable as a library.

What actually inspired me to do this was https://github.com/pdm-project/pdm

@tekumara tekumara merged commit 550b6a3 into master Jul 20, 2022
@tekumara tekumara deleted the pyproject-toml-pep621 branch July 20, 2022 23:59
@tekumara tekumara mentioned this pull request Jul 27, 2022
tekumara added a commit that referenced this pull request Jul 27, 2022
moving to pyproject.toml in #372 broke the tagger because `python setup.py --version` now returns `0.0.0`
@tekumara tekumara mentioned this pull request Jul 30, 2022
tekumara added a commit that referenced this pull request Jul 30, 2022
also fixes the build cache and toast.yml to use pyproject.toml introduced in #372
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants