Skip to content

Conversation

@talwrii
Copy link
Contributor

@talwrii talwrii commented Jan 6, 2023

Was erroring out because Iterator has moved to collections.abs.

@S0AndS0
Copy link
Member

S0AndS0 commented Jan 7, 2023

Thank you for catching and correcting this!

From what I could find Iterator was moved from collections to collections.abc around Python version 3.3, though I've yet to confirm that, and I'm a bit wary of breaking things for those on systems without a access to recent version of Python.

Any chance ya'd be willing to add something like the following to this Pull Request so we can maintain backwards compatibility?

import sys

if sys.version_info <= (3, 3):
    from collections import Iterator
else:
    from collections.abc import Iterator

@S0AndS0 S0AndS0 merged commit 67e94b7 into python-utilities:master Jan 14, 2023
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