-
Notifications
You must be signed in to change notification settings - Fork 9
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
chore: support Python 3.12 #430
Conversation
Signed-off-by: SdgJlbl <sarah.diot-girard@owkin.com>
@@ -22,7 +22,7 @@ Installation | |||
|
|||
.. _installation: | |||
|
|||
Substrafl and Substra are compatible with Python versions 3.9, 3.10 and 3.11 on Windows, MacOS and Linux. | |||
Substrafl and Substra are compatible with Python versions 3.9, 3.10, 3.11 and 3.12 on Windows, MacOS and Linux. |
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.
Do we still say that we are compatible with Python 3.9 if we are removing testing on it?
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.
We are still testing in Python 3.9 in the CI and in all other repos.
@@ -15,7 +15,7 @@ jobs: | |||
strategy: | |||
matrix: | |||
os: [ubuntu-22.04] | |||
python: ["3.8", "3.9", "3.10", "3.11"] | |||
python: ["3.10", "3.11", "3.12"] |
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.
Following @guilhem-barthes' comment, python 3.9 has been removed here
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.
It was intentional, since I realised that the documentation CI is still running in 3.8 as of today despite Python 3.8 support being dropped since March, and I was afraid that this would happen again for 3.9.
But I can also add a line in the Linear card instead and keep the CI running on 3.9 until that happens if you think it's better :)
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.
Fair enough, looks ok remove IMO!
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.
Thank you 📚
Adding support for Python 3.12, in particular in the CI.
Dropping 3.8 and 3.9 from the CI (both versions at the same time, since we forgot to do it last time).