Modernize python package setup#331
Open
Alicipy wants to merge 5 commits into
Open
Conversation
izar
approved these changes
May 21, 2026
Collaborator
Alicipy
force-pushed
the
modernize-python-package-setup
branch
from
3a02cec to
549244d
Compare
Author
|
Yes of course, I would love to! How do you suggest to proceed? Shall we merge this and redo the other one, or vice versa? This change does work in both, poetry and uv; as it depends only on PEP-specified behaviour right now. |
Alicipy
force-pushed
the
modernize-python-package-setup
branch
from
549244d to
973251f
Compare
Lets any packaging tool read the metadata, not just Poetry, and puts the fields as described in https://packaging.python.org/en/latest/specifications/pyproject-toml/#pyproject-toml-spec Signed-off-by: Stefan Kraus <dev@stefankraus.org>
[dependency-groups] is the standard table for these now Signed-off-by: Stefan Kraus <dev@stefankraus.org>
hatchling is the PyPA-maintained build backend; building a wheel no longer needs Poetry installed. Poetry can stay for the workflow. Signed-off-by: Stefan Kraus <dev@stefankraus.org>
This is done automatically as it's part of pytm package Signed-off-by: Stefan Kraus <dev@stefankraus.org>
Probably missed from a previous migration, most used repo lives at OWASP/pytm now. Signed-off-by: Stefan Kraus <dev@stefankraus.org>
Alicipy
force-pushed
the
modernize-python-package-setup
branch
from
973251f to
fccb68b
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
While starting with pytm, I saw a few things regarding package management that could use improvement, especially as the Python folks use pure pip or uv currently, and maybe wondering why the commands don't work.
This PR includes migration to the default pyproject.toml keys which poetry in version 2 also understands, so nothing breaks. As pip can handle these as well, we can remove a few other files which existed for compatibility reasons (?) as well.