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

Import Iterable from collections.abc for >=3.8. #39

Merged
merged 3 commits into from
Sep 5, 2019

Conversation

soasme
Copy link

@soasme soasme commented Apr 17, 2019

https://docs.python.org/3/library/collections.abc.html#collections.abc.Iterable

Below is the warning message triggered using any modules in fn.iters
from Python 3.7.

fn/iters.py:2: DeprecationWarning: Using or importing the ABCs
from 'collections' instead of from 'collections.abc' is
deprecated, and in 3.8 it will stop working

    from collections import deque, Iterable

-- Docs: https://docs.pytest.org/en/latest/warnings.html

https://docs.python.org/3/library/collections.abc.html#collections.abc.Iterable

Below is the warning message triggered using any modules in fn.iters
from Python 3.7.

    fn/iters.py:2: DeprecationWarning: Using or importing the ABCs
    from 'collections' instead of from 'collections.abc' is
    deprecated, and in 3.8 it will stop working

        from collections import deque, Iterable

    -- Docs: https://docs.pytest.org/en/latest/warnings.html
    ./fn/monad.py:51:44: W605 invalid escape sequence '\ '
    ./fn/monad.py:52:40: W605 invalid escape sequence '\ '
    ./fn/monad.py:53:41: W605 invalid escape sequence '\ '
Copy link

@jacobbridges jacobbridges left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! I'll try to resolve the migration conflicts and merge this in.

@jacobbridges jacobbridges merged commit 9958f1f into fnpy:master Sep 5, 2019
@jacobbridges
Copy link

@soasme Thanks for your work on this! I'll make sure this goes out with the next version.

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