Skip to content
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

Promise should not install typing for python 3.5+ #60

Open
brianbruggeman opened this issue Aug 1, 2018 · 0 comments
Open

Promise should not install typing for python 3.5+ #60

brianbruggeman opened this issue Aug 1, 2018 · 0 comments

Comments

@brianbruggeman
Copy link
Contributor

The pypi package for typing is a backport and not necessarily representative of the typing standard library for a given version of Python. For versions above Python 3.4, typing should not be installed as this can cause problems for Pythons 3.5+.

See: https://pypi.org/project/typing/

Code: https://github.com/syrusakbary/promise/blob/master/setup.py#L53

brianbruggeman added a commit to brianbruggeman/promise that referenced this issue Aug 13, 2018
Python 3.5 includes the typing module which doesn't appear to be
version compatible with the pip package of the same name.  Consequently,
the typing module is actually installed within a pip environment.  This
can cause problems for development when the pip version of the typing
module overrides the builtin version.

---------- coverage: platform darwin, python 3.7.0-final-0 -----------
Name                              Stmts   Miss  Cover
-----------------------------------------------------
promise/__init__.py                  11      0   100%
promise/async_.py                    84     10    88%
promise/dataloader.py               108      9    92%
promise/promise.py                  424     17    96%
promise/promise_list.py              84     13    85%
promise/pyutils/__init__.py           0      0   100%
promise/pyutils/version.py           42     18    57%
promise/schedulers/__init__.py        0      0   100%
promise/schedulers/asyncio.py        13     13     0%
promise/schedulers/gevent.py         14     14     0%
promise/schedulers/immediate.py      15      1    93%
promise/schedulers/thread.py         13     13     0%
-----------------------------------------------------
TOTAL                               808    108    87%

========== warnings summary ========================================
tests/test_awaitable_35.py::test_coroutine_is_thenable
  promise/tests/test_awaitable_35.py:26: RuntimeWarning: coroutine 'test_coroutine_is_thenable.<locals>.my_coroutine' was never awaited
    assert is_thenable(my_coroutine())

tests/test_extra.py::test_promises_promisify_still_works_but_deprecated_for_non_callables
  promise/promise.py:745: DeprecationWarning: Promise.promisify is now a function decorator, please use Promise.resolve instead.
    "Promise.promisify is now a function decorator, please use Promise.resolve instead."

-- Docs: http://doc.pytest.org/en/latest/warnings.html
=========== 114 passed, 9 skipped, 2 warnings in 2.26 seconds ======

Github issue syrusakbary#60
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

No branches or pull requests

1 participant