-
Notifications
You must be signed in to change notification settings - Fork 71
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
Remove python 3.8 support #972
Conversation
8e9b478
to
90230ca
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice 👍
Coverage reportClick to see where and how coverage changed
The report is truncated to 25 files out of 61. To see the full report, please visit the workflow summary page. This report was generated by python-coverage-comment-action |
pyupgrade can also be bumped after this: karapace/.pre-commit-config.yaml Line 37 in a4030fe
(Although, it would be nice to swap all of the tooling to ruff instead) |
@aiven-anton I'll change to |
90230ca
to
646fb93
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
About this change - What it does
Remove support for Python 3.8, it is EOL.