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

Minimum Python version 3.9 #1018

Merged
merged 2 commits into from
Aug 3, 2024
Merged

Conversation

legobeat
Copy link
Contributor

@legobeat legobeat commented Aug 3, 2024

Suggesting to drop advertised support for Python 3.7 and 3.8, increasing minimum supported version to 3.9.

  • Python 3.7 went EOL 2023-06-27 and is already untested.
  • Python 3.8 is nearing EOL (ETA 2024-10).
  • 3.9 matches distro versions of:
    • Debian bullseye (oldstable)
    • At least down to Fedora 37

(I'd personally go as far as suggesting bumping as far as 3.10 or even 3.11 (Debian stable/Fedora 38) as the type annotation features there could be nice. Starting with this more conservative change to take the temperature on how conservative podman-compose intends to be wrt runtime backwards-compatibility :))

@legobeat legobeat marked this pull request as ready for review August 3, 2024 00:55
@p12tic
Copy link
Collaborator

p12tic commented Aug 3, 2024

Agreed regarding upgrading to Python 3.9. Older distributions don't even have podman, so supporting older python is moot. If newer podman can be installed, then Python can be as well.

Regarding dropping Python 3.9 I think let's wait until it's EOL itself. The benefits of typing are small to a tiny project like podman-compose. The downsides of some commercial customer of Red Hat being unable to upgrade to get a bugfix they need are larger.

@p12tic p12tic merged commit b7514a0 into containers:main Aug 3, 2024
7 checks passed
@legobeat legobeat deleted the drop-python-3.8 branch August 3, 2024 14:58
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.

2 participants