-
Notifications
You must be signed in to change notification settings - Fork 69
Python packaging modernization #286
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
Conversation
dalcinl
commented
Nov 27, 2025
- Update to use scikit-build-core as the build backend.
- Ship the data/ directory in the wheels to make them self-contained.
|
Thanks, this is cool. It's been a while I used scikit-wheel. What's the metadata code for? |
Please note I switched to use the modern scikit-build-core, not the older scikit-build.
Do you mean the code in Right now the PyPI landing page is quite succinct https://pypi.org/project/mutationpp/, what this PR does, once a new version is published, is to basically render README.md instead. If you do not like all this complication, remove all this dynamic metadata stuff and keep the PyPI landing page minimal. After all, on the left side, PyPI will render the project URLs I'm adding, so users will know where to go for additional information. |
|
@rdbisme I'm planning to submit another PR to switch from pybind11 to nanobind. All of this work in in this branch: https://github.com/dalcinl/Mutationpp/tree/use-nanobind-abi3 Would you prefer to this PR with all that work? Or better keep things separate for easier review/discussion? |
9d7a3ce to
4fcbb17
Compare
|
@rdbisme I've updated this PR quite a bit. The failures in Python 3.8 are due to missing features in |
|
I'd say we can definitely drop 3.8 and 3.9. |
4fcbb17 to
564e430
Compare
564e430 to
c923071
Compare
|
@rdbisme This one is ready to go. Once merged, I plan to submit another one switching to
|
|
Yes already a maintainer and the wheel release I believe should be already wired when tagging. |
c923071 to
d53701e
Compare