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

associations: use collections.abc instead of collections #63

Merged
merged 2 commits into from
Jan 14, 2022

Conversation

miri64
Copy link
Contributor

@miri64 miri64 commented Dec 16, 2021

Using or importing the ABCs from collections instead of from collections.abc is deprecated since Python 3.3, and in 3.10 it will stop working. Since Python 3.10 was released a few month ago, and starting to get deployed in operating systems such as Arch or Fedora, this should be fixed ASAP.

@miri64 miri64 force-pushed the collections-fix branch 3 times, most recently from 5330444 to 8e29765 Compare December 17, 2021 13:17
@aabadie
Copy link
Contributor

aabadie commented Jan 14, 2022

Support for Python 2 has just been removed and the cli-tools are now supporting Python 3.7+. Please rebase. Maybe you could also add "3.10" in the GH action matrix ?

Using or importing the ABCs from `collections` instead of from
`collections.abc` is deprecated since Python 3.3, and in 3.10 it will
stop working. Since Python 3.10 was released a few month ago, and
starting to get deployed in operating systems such as Arch or Fedora,
this should be fixed ASAP.
@miri64
Copy link
Contributor Author

miri64 commented Jan 14, 2022

Support for Python 2 has just been removed and the cli-tools are now supporting Python 3.7+. Please rebase. Maybe you could also add "3.10" in the GH action matrix ?

Done and done.

@aabadie aabadie merged commit aa299c5 into iot-lab:master Jan 14, 2022
@aabadie
Copy link
Contributor

aabadie commented Jan 14, 2022

Thanks !

@miri64 miri64 deleted the collections-fix branch January 14, 2022 17:16
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.

None yet

2 participants