-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
12032 mypy 1.8 #12080
12032 mypy 1.8 #12080
Conversation
please review |
Thanks. Great job clening the "notes". I was looking for a way to turn notes into errors. I have pushed a commit that fixed the remaining issues.
|
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.
Many thanks.
Not sure about this change
it = it.addErrback(_handleCancelInlineCallbacks, status)
Other than that, it can be merged.
@@ -2116,14 +2116,13 @@ def _addCancelCallbackToDeferred( | |||
@param status: a L{_CancellationStatus} tracking the current status of C{gen} | |||
""" | |||
it.callbacks, tmp = [], it.callbacks | |||
it.addErrback(_handleCancelInlineCallbacks, status) | |||
it = it.addErrback(_handleCancelInlineCallbacks, status) |
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.
not sure what is going on here.
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.
It now makes sense. I have no idea why I have not observed it... Friday night, 11PM :(
This is necessary to back-propagate the type information about what the expected return value is into the type checker for |
Co-authored-by: Adi Roiban <adiroiban@gmail.com>
Co-authored-by: Adi Roiban <adiroiban@gmail.com>
Thanks for that. This didn't show up on my local runs for some reason, which is interesting, but glad to clear those out of CI |
@adiroiban it still needs a confirming review :) |
please review |
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.
All good.
Make sense.
Scope and purpose
Fixes #12032