Skip to content

Update text to recommend Python v3.10+#2013

Open
tomodwyer wants to merge 1 commit into
mainfrom
tomodwyer/python-versions
Open

Update text to recommend Python v3.10+#2013
tomodwyer wants to merge 1 commit into
mainfrom
tomodwyer/python-versions

Conversation

@tomodwyer

@tomodwyer tomodwyer commented Jul 23, 2026

Copy link
Copy Markdown
Member

As we have updated OpenSAFELY CLI to drop support for Python v3.8 and v3.9

opensafely-core/opensafely-cli#280

@tomodwyer tomodwyer self-assigned this Jul 23, 2026
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 23, 2026

Copy link
Copy Markdown

Deploying opensafely-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: b8a130d
Status: ✅  Deploy successful!
Preview URL: https://b56b1c45.opensafely-docs.pages.dev
Branch Preview URL: https://tomodwyer-python-versions.opensafely-docs.pages.dev

View logs

@mikerkelly mikerkelly left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks Thomas.

The supported opensafely-cli versions are now 3.10, 3.11, 3.12, and 3.13. that does not include Python 3.14, versions of which have been released since 2025-10.

Given that, I think it would be more helpful to state explicitly that versions between 3.10--3.13 are supported. As if someone has the most recent Python, 3.14, as we recommend, they will not be able to use it right now. That could be frustrating, because we told them something not true, and depending on their control over their system the install may not have been a trivial thing to do (for example, if they had to seek special permission to have 3.14 installed, only to find that this does not work, and they have to go back and ask again).

What do you think?

I think that is relevant to docs/install-python.md, and also README.md from opensafely-cli which recommends "at least version 3.10 but the more recent the better". I think that we should change both of those.

I don't think we should change docs/index.md, see comment.

Comment thread docs/index.md Outdated
@tomodwyer

Copy link
Copy Markdown
Member Author

@mikerkelly I think I was being optimistic in hoping the Python v3.14 changes would be done before we would need to merge this, but you're right I'll add another PR in for that after this one.

As we have updated OpenSAFELY CLI to drop support for Python v3.8 and
v3.9

opensafely-core/opensafely-cli#280
@mikerkelly

Copy link
Copy Markdown
Contributor

@mikerkelly I think I was being optimistic in hoping the Python v3.14 changes would be done before we would need to merge this, but you're right I'll add another PR in for that after this one.

Fair. I think that any "version X+" when we actually require a bounded range is hostage to any future release making it inaccurate without us noticing. So it might be best to always state the exact range. Would be nice if we could somehow configure that once as a variable and have it propagate in the docs, probably not worth it only if very quick though.

@tomodwyer

Copy link
Copy Markdown
Member Author

Good point, we don't automatically support 3.15 when it's out.

@tomodwyer
tomodwyer requested a review from mikerkelly July 23, 2026 14:16
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