-
Notifications
You must be signed in to change notification settings - Fork 28
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
Fix typing error under mypy 0.990 #1156
Conversation
didn't check what failures are due to. is it the same as dandi/dandi-archive#1365 but even in the container we use? |
@yarikoptic No, this is a type-checking error. It has absolutely nothing to do with the API server performance. |
oh right , FTR looked into one of the errors and it is
|
Codecov ReportBase: 88.29% // Head: 66.99% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #1156 +/- ##
===========================================
- Coverage 88.29% 66.99% -21.30%
===========================================
Files 73 73
Lines 8800 8651 -149
===========================================
- Hits 7770 5796 -1974
- Misses 1030 2855 +1825
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
@yarikoptic Now all the remaining test failures are ones that will be resolved by #1157. |
The latest release of mypy now detects an error in our type annotations, which this PR fixes.