Skip to content

Conversation

@Cito
Copy link
Member

@Cito Cito commented Sep 22, 2025

The commons lib already supported Python 3.13, I just added the classifier and updated the base image to use Python 3.13 as well, and the latest Debian. Needed to deactivate Moby because it is not supported there. Also, switched uivcorn from websockets (default) to the newer websockets-sansio. We're currently not using it, but it also avoids deprecation warnings.

Also un-supported Python 3.9 since some of our deps do not support it any more in their latest version. With our single lock file approach we would need to make concessions and use older versions of deps just to accomodate Python 3.9. Maybe we should test older Python versions without lock file. But Python 3.9 has reached its EOL anyway and does not even get security patches any more, so we should not support it any more either.

@coveralls
Copy link

coveralls commented Sep 22, 2025

Pull Request Test Coverage Report for Build 17938489613

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 91.81%

Totals Coverage Status
Change from base Build 16443508357: 0.02%
Covered Lines: 852
Relevant Lines: 928

💛 - Coveralls

TheByronHimes
TheByronHimes previously approved these changes Sep 22, 2025
mephenor
mephenor previously approved these changes Sep 22, 2025
@Cito Cito dismissed stale reviews from mephenor and TheByronHimes via d889776 September 23, 2025 07:11
@Cito
Copy link
Member Author

Cito commented Sep 23, 2025

Sorry, added 4 more commits, can you re-approve?

@Cito Cito removed the request for review from mephenor September 23, 2025 07:26
@Cito Cito merged commit b50e46e into main Sep 23, 2025
5 checks passed
@Cito Cito deleted the feature/support-python313-gsi-1854 branch September 23, 2025 07:27
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.

5 participants