-
Notifications
You must be signed in to change notification settings - Fork 10
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 to development with Poetry #45
Conversation
Ignore the tests github action for now, I will change these in subsequent PRs. |
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.
I'd start with a PR that updates the CI test workflow to run on Python 3.10.
I can create a separate PR first that updates the CI to test if everything works on python 3.10 (or higher). But ultimately I do plan to replace the tests completely. |
Yes, replacing them is ok, but it's good to have each step of the way tested and running in CI. |
Do you want to keep the minimum version at 3.10 or set it to 3.11 (actually my preference)? In all my other packages I have set 3.11. |
3.11 is fine. 👍 |
0301afc
to
b90bcc2
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.
Thanks!
I'd like to introduce Poetry to this project, it works great and helps a lot on code quality.
In this first PR I only focused on the basic things such as the package code, examples and the pyproject.toml config.
This PR will also change to minimal python version from 3.7 to 3.11.