-
-
Notifications
You must be signed in to change notification settings - Fork 2.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
Drop explicit loop #3580
Drop explicit loop #3580
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3580 +/- ##
==========================================
- Coverage 97.79% 97.74% -0.05%
==========================================
Files 43 43
Lines 8653 8617 -36
Branches 1385 1378 -7
==========================================
- Hits 8462 8423 -39
- Misses 81 83 +2
- Partials 110 111 +1
Continue to review full report at Codecov.
|
This pull request introduces 6 alerts when merging b9d88ff into 6fa0be8 - view on LGTM.com new alerts:
|
This pull request fixes 1 alert when merging d12719b into 6fa0be8 - view on LGTM.com fixed alerts:
|
This pull request fixes 1 alert when merging 50d306e into 6fa0be8 - view on LGTM.com fixed alerts:
|
This pull request fixes 1 alert when merging 99e839d into 79364ce - view on LGTM.com fixed alerts:
|
This pull request fixes 1 alert when merging c58a0e6 into e1eb27a - view on LGTM.com fixed alerts:
|
Use
asyncio.get_event_loop()
instead if the loop instance is needed.All aiohttp objects work with the currently running loop, a creation of aiohttp instances, e.g.
ClientSession
when the loop is not running is forbidden.As a side effect of PR passing callables for
aiohttp_server()
andaiohttp_client
pytest fixtures are forbidden, please call these callables explicitly.