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

Drop Python 3.9 support, require 3.10+ #353

Merged
merged 1 commit into from
May 24, 2024
Merged

Drop Python 3.9 support, require 3.10+ #353

merged 1 commit into from
May 24, 2024

Conversation

EwoutH
Copy link
Collaborator

@EwoutH EwoutH commented Mar 27, 2024

Following SPEC 0 Python 3.9 support is dropped for future minor and major releases, and will require Python 3.10+ from version 3.0 onwards.

The main benefit of requiring Python 3.10 is that we can implement type hinting more elegant in the workbench.

The 2.5.x branch will keep supporting Python 3.9.

Following SPEC 0 Python 3.9 support is dropped for future minor and major releases, and will require Python 3.10+ from version 3.0 onwards.

The main benefit of requiring Python 3.10 is that we can implement type hinting more elegant in the workbench.

The 2.5.x branch will keep supporting Python 3.9.
@EwoutH EwoutH requested a review from quaquel March 27, 2024 19:30
@EwoutH EwoutH added this to the 3.0 milestone Mar 27, 2024
@coveralls
Copy link

Coverage Status

coverage: 80.3%. remained the same
when pulling b0ecdb9 on python-3.10
into 10429e4 on master.

@quaquel
Copy link
Owner

quaquel commented Mar 28, 2024

looks fine to me. Just one quick question (which I can look up myself once I have time). What is the default python version on DelftBlue?

@EwoutH
Copy link
Collaborator Author

EwoutH commented Mar 28, 2024

Default 3.9, with 3.10 pre-installed. I think it will be updated around summer again.

@EwoutH
Copy link
Collaborator Author

EwoutH commented Mar 28, 2024

We can backport all MPI changes to the 2.5 branch to maintain full compatibility.

@quaquel
Copy link
Owner

quaquel commented Mar 28, 2024

Ok, let me try and finish the mpi bugfix branch first (testing atm), so I don't need to worry about 3.9/3.10.

@EwoutH
Copy link
Collaborator Author

EwoutH commented May 24, 2024

Can we merge this one?

@quaquel
Copy link
Owner

quaquel commented May 24, 2024

I was wondering the same thing, but yes LGTM.

@EwoutH EwoutH merged commit 5b33698 into master May 24, 2024
20 checks passed
@EwoutH EwoutH deleted the python-3.10 branch May 24, 2024 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants