-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Additional type information for asyncio #81
Conversation
Thanks! Could you fix the issues discovered by Travis CI? |
I am checking on the error |
Add the appropriate types for AbstractServer in asyncio.events. wait_closed() is a couroutine, however in the implementation of AbstractServer not marked as such. We are adding the couroutine defintion here anyway, as we do want to make it typeable as a coroutine if necessary.
Try something like |
This seems to have been caused by absolute imports having mypy's internal typeshed information pulled in, which conflicts. @gvanrossum thanks for the hint, much appreciated. |
Additional type information for asyncio
Thanks again! |
This pull request adds more type information for asyncio.
I've tested this against a small code library internally that was written in Python 3.5 and uses asyncio.subprocess. I tested this with mypy at 9befcc7.
I am sure some of the types need refinement as this is the first time I am working with typeshed and typed python.