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

chore: support Python 3.12 #430

Merged
merged 1 commit into from
Jul 25, 2024
Merged

chore: support Python 3.12 #430

merged 1 commit into from
Jul 25, 2024

Conversation

SdgJlbl
Copy link
Contributor

@SdgJlbl SdgJlbl commented Jul 24, 2024

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).

Signed-off-by: SdgJlbl <sarah.diot-girard@owkin.com>
@SdgJlbl SdgJlbl requested a review from a team as a code owner July 24, 2024 14:57
@@ -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.
Copy link
Contributor

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?

Copy link
Contributor Author

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"]
Copy link
Contributor

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

Copy link
Contributor Author

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 :)

Copy link
Contributor

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!

Copy link
Contributor

@thbcmlowk thbcmlowk left a comment

Choose a reason for hiding this comment

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

Thank you 📚

@SdgJlbl SdgJlbl merged commit 019ec91 into main Jul 25, 2024
6 checks passed
@SdgJlbl SdgJlbl deleted the chore/python3.12 branch July 25, 2024 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants