Skip to content

Remove support for Python 3.7 #1717

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

miguelgrinberg
Copy link
Collaborator

As discussed, we are removing Python 3.7 from the actively supported versions.

@miguelgrinberg miguelgrinberg requested a review from pquentin March 19, 2024 14:21
Copy link
Member

@pquentin pquentin left a comment

Choose a reason for hiding this comment

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

Thanks! LGTM. I thought this would be more involved than that.

Can you please run pip install pyupgrade && pyupgrade --py38-plus **/*.py? This will update build-dist.py to use shlex.join (new in Python 3.8) and python -m build (as invoking setup.py is deprecated).

You also need to change py37 to py38 in noxfile.py (and make sure to do it in utils/run-unasync.py in the async PR). Not that it makes any difference.

Also, yay, we can now use the walrus operator, positional-only parameters and use = in f-strings! https://docs.python.org/3/whatsnew/3.8.html

@pquentin pquentin mentioned this pull request Mar 20, 2024
@miguelgrinberg miguelgrinberg merged commit ab1421a into elastic:main Mar 20, 2024
@miguelgrinberg miguelgrinberg deleted the remove-py37 branch March 20, 2024 11:38
miguelgrinberg added a commit to miguelgrinberg/elasticsearch-dsl-py that referenced this pull request Apr 3, 2024
@miguelgrinberg miguelgrinberg added the backport 8.x Backport to 8.x label Apr 3, 2024
github-actions bot pushed a commit that referenced this pull request Apr 3, 2024
miguelgrinberg added a commit that referenced this pull request Apr 3, 2024
(cherry picked from commit ab1421a)

Co-authored-by: Miguel Grinberg <miguel.grinberg@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
backport 8.x Backport to 8.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants