Skip to content
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

Merged
merged 9 commits into from
Apr 26, 2024
Merged

Migrate to development with Poetry #45

merged 9 commits into from
Apr 26, 2024

Conversation

klaasnicolaas
Copy link
Contributor

@klaasnicolaas klaasnicolaas commented Apr 26, 2024

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.

⚠️ Breaking change! ⚠️

This PR will also change to minimal python version from 3.7 to 3.11.

@klaasnicolaas klaasnicolaas added refactor Improvement of existing code, not introducing new features. maintenance Generic maintenance tasks. labels Apr 26, 2024
@klaasnicolaas
Copy link
Contributor Author

Ignore the tests github action for now, I will change these in subsequent PRs.

Copy link
Contributor

@MartinHjelmare MartinHjelmare left a 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.

pyproject.toml Outdated Show resolved Hide resolved
@klaasnicolaas
Copy link
Contributor Author

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.

@MartinHjelmare
Copy link
Contributor

Yes, replacing them is ok, but it's good to have each step of the way tested and running in CI.

@klaasnicolaas
Copy link
Contributor Author

klaasnicolaas commented Apr 26, 2024

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.

@MartinHjelmare
Copy link
Contributor

3.11 is fine. 👍

pyproject.toml Outdated Show resolved Hide resolved
@klaasnicolaas klaasnicolaas added the breaking-change A breaking change for existing users. label Apr 26, 2024
Copy link
Contributor

@MartinHjelmare MartinHjelmare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@klaasnicolaas klaasnicolaas merged commit 6e53f8e into master Apr 26, 2024
4 checks passed
@klaasnicolaas klaasnicolaas deleted the klaas-2024-005 branch April 26, 2024 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change A breaking change for existing users. maintenance Generic maintenance tasks. refactor Improvement of existing code, not introducing new features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants