-
Notifications
You must be signed in to change notification settings - Fork 77
Updating all type annotations to >3.10 conventions #1246
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
Conversation
f1186df
to
dde2ff0
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #1246 +/- ##
==========================================
- Coverage 89.55% 89.53% -0.03%
==========================================
Files 150 150
Lines 15368 15303 -65
==========================================
- Hits 13763 13701 -62
+ Misses 1605 1602 -3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
dde2ff0
to
7168a54
Compare
remove 'Optional' annotations update 'List' annotations to 'list' update 'Dict' annotations to 'dict' update 'Tuple' annoatations to 'tuple' Remove 'Union' annotations
differs vs. 'typing._GenericAlias'
7168a54
to
41ab593
Compare
just a reminder on this one @tschaume |
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! The linebreaks for the | None
are a little off I think. But that's nothing major.
ripping the bandaid off -_-
making diffs easier to parse in #1243