Modernize Python tooling#36
Conversation
bigjools
left a comment
There was a problem hiding this comment.
I filed the ticket yesterday so I had something to work on today :(
This change is doing WAY too much all at once; the intention of the ticket was to only replace setup.py with pyproject.toml, and switch the build system to hatchling and setuptools_scm. I specifically did not want to remove tox (yet) nor change linters, the publisher action, or anything like that. (Side note: you have not updated any documentation.)
It's not really that much change, it just looks like it. The Most of what looks like churn is simply copy+paste of discrete config files into the corresponding The linter changes are similarly low-effort; my original approach was to drop in Ruff with a giant stack of ignores, and promise to peel them back later. I can do that instead, if it makes things easier. |
|
Downsized for the sake of sanity. |
45aa1b4 to
3e5a727
Compare
bigjools
left a comment
There was a problem hiding this comment.
This is going to have ramifications on the release procedure as it will now react to the tag rather than the version in const.py, I'll check it all out when I get some time unless Mr Le Manchet beats me to it.
Move all configs into `pyproject.toml`, switch test/packaging to use Hatch, and do all the linting with Ruff. Fix issues that Ruff turned up, and annotate/ignore the pieces that it got wrong. Fixes: Issue NicolasLM#33
Generate the documentation with `hatch run docs:build $TYPE`
|
I may close this and re-do it using UV exclusively, as it avoids some of the Hatch foibles. |
|
A review from Claude below. It can serve as a basis for the transition to a pure UV project. IssuesMissing Python 3.13 classifier — The old setup.py and the CI matrix both include 3.13, but the new Typo:
Coverage file location — The CI still relies on Minor
|
|
This has been sitting around for so long, it now needs its own modernisation effort. Closing. |
Move all configs into
pyproject.toml, switch test/packaging to use Hatch, and do all the linting with Ruff.Fix issues that Ruff turned up, and annotate/ignore the pieces that it got wrong.
Fixes: Issue #33