-
Notifications
You must be signed in to change notification settings - Fork 5
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
Make it compatible with PEP-567 #2
Comments
In c33d9fc I replaced the local implementation with upstream |
@fantix will you be cutting a 0.2.0 release soon? |
Yes! Will do as soon as test is fixed. |
Do you mean the failing travis build for master or some specific test failures? I believe there is an issue with python 3.7 in Travis CI's default configuration -- more specifically with Python 3.7 and Ubuntu 14.04. travis-ci/travis-ci#9815 describes a workaround. If you update .travis.yml to contain something like this, the build may pass. python:
- 3.5
- 3.6
matrix:
include:
- python: 3.7
dist: xenial
sudo: true I'll go ahead and open a PR with this change. |
Yes that’s the one you fixed. Thanks a lot! I’ll push a release as soon as I’m back to my keyboard. |
Closing this now. We won't add the keyword argument |
PEP-567 is accepted for Python 3.7.
We should make this library compatible.contextvars should replace this library.The text was updated successfully, but these errors were encountered: