-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
async_cluster: add/update typing #2195
Conversation
c1c85ec
to
ca09cb9
Compare
Codecov Report
@@ Coverage Diff @@
## master #2195 +/- ##
==========================================
+ Coverage 91.99% 92.00% +0.01%
==========================================
Files 108 108
Lines 27380 27390 +10
==========================================
+ Hits 25189 25201 +12
+ Misses 2191 2189 -2
Continue to review full report at Codecov.
|
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 as always @utkarshgupta137! I just merged in the latest master- it all passes, and we’ll merge this in.
@dvora-h Your merge commit has removed |
Sorry, I fixed that |
5f86d98
to
1f9e0ba
Compare
1f9e0ba
to
ebf2ded
Compare
@utkarshgupta137 Thanks for the quick fix. |
Pull Request check-list
$ tox
pass with this change (including linting)?