Skip to content

Replace python2 deprecation with a badge of supported python versions #12199

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 1 commit into from
Oct 1, 2023
Merged

Replace python2 deprecation with a badge of supported python versions #12199

merged 1 commit into from
Oct 1, 2023

Conversation

jeffwidman
Copy link
Contributor

The python world has (mostly) moved on from Python 2. Anyone not already aware of the py2->py3 migration is probably new to the ecosystem and started on Python 3.

Additionally, it's convenient to see at a glance what versions of Python are supported by the current release. This pulls from PyPI versions, so will not immediately match main (we can probably change this to match main if preferred).

So by doing this it's both more useful going forward, and also lets us drop the explicit notice about dropping Python 2 support.

Again, I skipped a news fragment as this seems too trivial to merit it.

The python world has (mostly) moved on from Python 2. Anyone not already aware of the py2->py3 migration is probably new to the ecosystem and started on Python 3.

Additionally, it's convenient to see at a glance what versions of Python are supported by the current release. This pulls from PyPI versions, so will not immediately match `main` (we can probably change this to match `main` if preferred).

So by doing this it's both more useful going forward, and also lets us drop the explicit notice about dropping Python 2 support.
@uranusjr uranusjr added the skip news Does not need a NEWS file entry (eg: trivial changes) label Aug 3, 2023
@jeffwidman
Copy link
Contributor Author

Any update on this? Since it's so trivial, be great to get a quick 👍 / 👎 on it and then be done with the decision.

Copy link
Member

@uranusjr uranusjr left a comment

Choose a reason for hiding this comment

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

The Python 2 bug reports have been far between recently, and anyone still reporting them is probably not going to be helped by a note in the README anyway.

@jeffwidman
Copy link
Contributor Author

Given this has 1 approval, what's still needed to merge it?

@jeffwidman
Copy link
Contributor Author

Anything I can do to help move this forward?

@sbidoul sbidoul merged commit 71df02c into pypa:main Oct 1, 2023
@jeffwidman jeffwidman deleted the patch-1 branch October 1, 2023 15:44
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
skip news Does not need a NEWS file entry (eg: trivial changes)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants