-
Notifications
You must be signed in to change notification settings - Fork 15
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
Use pyproject.toml
for setuptools config
#55
Use pyproject.toml
for setuptools config
#55
Conversation
pyproject.toml
Outdated
@@ -2,6 +2,34 @@ | |||
requires = ["setuptools>=42", "wheel", "setuptools_scm[toml]~=6.0"] | |||
build-backend = "setuptools.build_meta" | |||
|
|||
[project] | |||
name = "diepvries" | |||
version = "0.9.2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be a dynamic attribute, as the version is computed with git tags and setuptools_scm
, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, definitely. I also bumped the minimum setuptools and setuptools_scm version as per https://github.com/pypa/setuptools_scm?tab=readme-ov-file#pyprojecttoml-usage
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you also update the information in CONTRIBUTING.md
which references setup.py
?
7f2edd0
to
a551167
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works like a charm for me! Good to go
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
Use
pyproject.toml
for setuptools config