Skip to content

Commit

Permalink
Merge pull request #1339 from pquentin/coverage-py38
Browse files Browse the repository at this point in the history
Enable Python 3.8 coverage
  • Loading branch information
njsmith authored Feb 3, 2020
2 parents 3c073d6 + 40df149 commit 1794202
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 6 deletions.
6 changes: 1 addition & 5 deletions ci.sh
Original file line number Diff line number Diff line change
Expand Up @@ -267,14 +267,10 @@ else
netsh winsock reset
fi

# Disable coverage on 3.8 until we run 3.8 on Windows CI too
# https://github.com/python-trio/trio/pull/784#issuecomment-446438407
if [[ "$(python -V)" = Python\ 3.8* ]]; then
true;
# coverage is broken in pypy3 7.1.1, but is fixed in nightly and should be
# fixed in the next release after 7.1.1.
# See: https://bitbucket.org/pypy/pypy/issues/2943/
elif [[ "$TRAVIS_PYTHON_VERSION" = "pypy3" ]]; then
if [[ "$TRAVIS_PYTHON_VERSION" = "pypy3" ]]; then
true;
else
# Flag pypy and cpython coverage differently, until it settles down...
Expand Down
5 changes: 4 additions & 1 deletion trio/_util.py
Original file line number Diff line number Diff line change
Expand Up @@ -72,10 +72,13 @@ def signal_raise(signum):
if sys.version_info < (3, 5, 2):

def aiter_compat(aiter_impl):
@wraps(aiter_impl)
# de-sugar decorator to fix Python 3.8 coverage issue
# https://github.com/python-trio/trio/pull/784#issuecomment-446438407
async def __aiter__(*args, **kwargs):
return aiter_impl(*args, **kwargs)

__aiter__ = wraps(aiter_impl)(__aiter__)

return __aiter__
else:

Expand Down

0 comments on commit 1794202

Please sign in to comment.