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

Bump min supported Python to 3.10 #992

Merged
merged 13 commits into from
Sep 26, 2024
Merged

Bump min supported Python to 3.10 #992

merged 13 commits into from
Sep 26, 2024

Conversation

janosh
Copy link
Member

@janosh janosh commented Sep 25, 2024

following suit with pymatgen which in turn follows numpy, scipy and the rest of the NumFOCUS stack, this PR drops Python 3.9 support.

@utf curious to hear your thoughts. we may want to add 3.12 to CI in the same PR. another open question is whether we want to set zip's new strict keyword to True. it defaults to False which preserves the old behavior of stopping whenever the 1st iterator is exhausted. True instead throws an error if all iterators don't have the same length

@janosh janosh added pkg Package health and distribution related stuff ci Continuous integration labels Sep 25, 2024
@janosh janosh requested review from utf and JaGeo September 25, 2024 14:51
@janosh janosh added the docs Improvements or additions to documentation label Sep 25, 2024
@utf
Copy link
Member

utf commented Sep 26, 2024

It's already time for a new conda env 😨. Thanks for this @janosh. It makes sense to follow suit with pymatgen.

Happy to add the strict option.

@@ -5,6 +5,7 @@
[![pypi version](https://img.shields.io/pypi/v/atomate2?color=blue)](https://pypi.org/project/atomate2)
![supported python versions](https://img.shields.io/pypi/pyversions/atomate2)
Copy link
Contributor

Choose a reason for hiding this comment

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

Here is a second python badge still listing 3.9.

Copy link
Member Author

Choose a reason for hiding this comment

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

that badge pulls the data it displays from PyPI, so will auto-update once this PR is merged and released

Copy link
Contributor

Choose a reason for hiding this comment

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

Oki, good to know :)

@janosh janosh enabled auto-merge (squash) September 26, 2024 13:30
@janosh
Copy link
Member Author

janosh commented Sep 26, 2024

@utf the required CI runs for auto-merging need updating. 3.9 is still required but should be replaced with 3.12

edit: nvm, already done

@janosh janosh merged commit 96b2b82 into main Sep 26, 2024
9 checks passed
@janosh janosh deleted the py310 branch September 26, 2024 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous integration docs Improvements or additions to documentation pkg Package health and distribution related stuff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants