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

python: drop support for 3.7 #432

Merged
merged 1 commit into from
Jul 3, 2023
Merged

Conversation

fscherf
Copy link
Member

@fscherf fscherf commented Jul 3, 2023

Since 27. Jun 2023, Python 3.7 is end of life (https://endoflife.date/python). Thus, this patch removes support for this version.

@fscherf fscherf force-pushed the fscherf/remove-python37-support branch from fff0246 to 4f0ead3 Compare July 3, 2023 12:05
Copy link
Member

@sobolevn sobolevn left a comment

Choose a reason for hiding this comment

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

You can also run pyupgrade if you wish (. It sometimes does good things to your code.

And don't forget to change mypy: https://github.com/lona-web-org/lona/blob/master/mypy.ini#L3

Since 27. Jun 2023, Python 3.7 is end of life
(https://endoflife.date/python). Thus, this patch removes support for this
version.

Signed-off-by: Florian Scherf <mail@florianscherf.de>
@fscherf fscherf force-pushed the fscherf/remove-python37-support branch from 4f0ead3 to f632500 Compare July 3, 2023 15:17
@fscherf
Copy link
Member Author

fscherf commented Jul 3, 2023

You can also run pyupgrade if you wish (. It sometimes does good things to your code.

Nice tool! I ran it and it seems to be happy with the code.

And don't forget to change mypy: https://github.com/lona-web-org/lona/blob/master/mypy.ini#L3

Good call. Fixed.

@fscherf
Copy link
Member Author

fscherf commented Jul 3, 2023

@sobolevn: Thanks for reviewing!

@fscherf fscherf merged commit d0b50b9 into master Jul 3, 2023
5 checks passed
@fscherf fscherf deleted the fscherf/remove-python37-support branch July 3, 2023 15:32
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