Skip to content

Adjust type of iscoroutine() argument#5729

Merged
webknjaz merged 1 commit intoaio-libs:masterfrom
JelleZijlstra:patch-1
May 22, 2021
Merged

Adjust type of iscoroutine() argument#5729
webknjaz merged 1 commit intoaio-libs:masterfrom
JelleZijlstra:patch-1

Conversation

@JelleZijlstra
Copy link
Contributor

This will be needed to make mypy pass after python/typeshed#5517 is released. The function accepts all objects, so object is the appropriate annotation, but Any is necessary for now here to make the conditional definition work with the previous type annotation.

What do these changes do?

Minor change in type annotation

Are there changes in behavior for the user?

No

Related issue number

I don't think this change requires an issue or changelog entry, but happy to provide one if you disagree.

This will be needed to make mypy pass after python/typeshed#5517 is released. The function accepts all objects, so `object` is the appropriate annotation, but `Any` is necessary for now here to make the conditional definition work with the previous type annotation.
@JelleZijlstra JelleZijlstra requested a review from asvetlov as a code owner May 21, 2021 23:33
@webknjaz webknjaz enabled auto-merge (squash) May 22, 2021 13:35
@codecov
Copy link

codecov bot commented May 22, 2021

Codecov Report

Merging #5729 (374a86d) into master (63eb6ff) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5729   +/-   ##
=======================================
  Coverage   97.17%   97.17%           
=======================================
  Files          41       41           
  Lines        8865     8865           
  Branches     1425     1425           
=======================================
  Hits         8615     8615           
  Misses        133      133           
  Partials      117      117           
Flag Coverage Δ
unit 97.06% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
aiohttp/helpers.py 96.83% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 63eb6ff...374a86d. Read the comment docs.

@webknjaz webknjaz merged commit dd72382 into aio-libs:master May 22, 2021
@patchback

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants