-
Notifications
You must be signed in to change notification settings - Fork 21
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
Migrate from setup.py to pyproject #43
Conversation
d7fd863
to
659d901
Compare
@dmulder do you prefer setuptools or pdm? |
I've never even heard of pdm. I have no preference. |
There is one issue I still haven't found out how to fix:
I don't see how to do that, you can't seem to provide a destination to all files. |
ed83762
to
bd27b11
Compare
bd27b11
to
2bdb0dd
Compare
7487843
to
298a88c
Compare
298a88c
to
c4ebc71
Compare
I got it sorted out. I also added a tox.ini filed and use gh-actions for running tox now. |
Check the tests output, it runs test for several python versions now. |
We could also enable linting and type checking if you want. |
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.
LGTM!
@cryptomilk I granted you write access to the repo, so feel free to merge when ready. |
Migrate from setup.py to pyproject.toml