-
-
Notifications
You must be signed in to change notification settings - Fork 371
Drop 3.9 support #3345
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
Drop 3.9 support #3345
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3345 +/- ##
================================================
Coverage 100.00000% 100.00000%
================================================
Files 125 125
Lines 19598 19388 -210
Branches 1344 1319 -25
================================================
- Hits 19598 19388 -210
🚀 New features to boost your workflow:
|
for more information, see https://pre-commit.ci
- Fix a few typos - Remove unnecessary generic function infrastructure - Deduplicate `from typing import ...` imports - Fix codecov complains
I'm not so sure about the unix pipes change. I suppose we'll get comments if it doesn't work? I can't tell why the code was there to begin with, which is a bad sign for being able to tell whether it's necessary anymore.
|
Planning on merging this in a few days unless people have concerns. |
jakkdl
left a comment
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.
looks good! Did not catch anything of note when reading through
This PR also notes that we support 3.14 now.