Skip to content

drop support for Python 3.8 in the LocalStack CLI #11875

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

Merged
merged 2 commits into from
Nov 19, 2024
Merged

Conversation

alexrashed
Copy link
Member

@alexrashed alexrashed commented Nov 19, 2024

Motivation

With v4 we will be dropping the support for Python 3.8 in our LocalStack CLI.
It reached its End Of Life on the 7th of October 2024 and does not get any updates anymore (not even security).

Changes

Similar to #9148, this PR bumps the minimum version in the pyproject.toml (+ in the ruff config), and removes 3.8 from the CLI test build matrix and adds 3.13 instead.

@alexrashed alexrashed added area: cli LocalStack CLI semver: major Breaking changes which can be included in a major release only labels Nov 19, 2024
@alexrashed alexrashed added this to the 4.0 milestone Nov 19, 2024
@alexrashed alexrashed self-assigned this Nov 19, 2024
@localstack localstack deleted a comment from localstack-bot Nov 19, 2024
@localstack localstack deleted a comment from localstack-bot Nov 19, 2024
@localstack-bot

This comment was marked as outdated.

Copy link

S3 Image Test Results (AMD64 / ARM64)

  2 files  ±0    2 suites  ±0   4m 5s ⏱️ - 1m 13s
421 tests ±0  369 ✅ ±0   52 💤 ±0  0 ❌ ±0 
842 runs  ±0  738 ✅ ±0  104 💤 ±0  0 ❌ ±0 

Results for commit 98cd310. ± Comparison against base commit e868002.

Copy link
Contributor

@bentsku bentsku left a comment

Choose a reason for hiding this comment

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

LGTM for the code change, but might want a sign-off from stakeholders 😄

Comment on lines +337 to +340
with (
self._s3_store.open(src_bucket, src_s3_object, mode="r") as src_stored_object,
self.open(s3_part, mode="w") as stored_part,
):
Copy link
Contributor

Choose a reason for hiding this comment

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

I was super curious about this, so here it is for history:
https://www.blog.pythonlibrary.org/2021/09/08/python-3-10-parenthesized-context-managers/

Super neat! 👌

Copy link

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 43m 52s ⏱️ +41s
3 528 tests ±0  3 135 ✅ ±0  393 💤 ±0  0 ❌ ±0 
3 530 runs  ±0  3 135 ✅ ±0  395 💤 ±0  0 ❌ ±0 

Results for commit 98cd310. ± Comparison against base commit e868002.

@alexrashed alexrashed merged commit bc4828f into master Nov 19, 2024
38 of 40 checks passed
@alexrashed alexrashed deleted the drop-python-3.8 branch November 19, 2024 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: cli LocalStack CLI semver: major Breaking changes which can be included in a major release only
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants