- Notifications
You must be signed in to change notification settings - Fork673
refactor: use more python3.9 syntax#3010
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
JohnVillalovos commentedOct 8, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Cool! I was thinking about this too 🙂 Well thinking of stopping the use of Thanks for doing this! Once we (in reality you) get this done, we should do a follow-up adding the commit to |
I was thinking of that as well, especially now that I see how big the diff is 😅 Maybe after this PR in case there's more rebasing etc. I'll just need to tweak the type hint tests for this now. |
I'll rebase this after#3083 to avoid too many conflicts there, and that will likely fix the typing issues altogether. |
@JohnVillalovos this one's a chunky one now, but mostly auto-generated 😅 |
codecovbot commentedFeb 5, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@## main #3010 +/- ##==========================================+ Coverage 97.25% 97.28% +0.03%========================================== Files 97 97 Lines 5932 5975 +43 ==========================================+ Hits 5769 5813 +44+ Misses 163 162 -1
Flags with carried forward coverage won't be shown.Click here to find out more.
|
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.
Thanks@nejch
A lot of work there.
4e90c11
intomainUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Follow-up to#3005
Mostly done with
pyupgrade --py39-plus
and some manual cleanup.