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

Drop outdated python 3.6, add support for python 3.10 #107

Merged
merged 2 commits into from
Aug 1, 2022
Merged

Drop outdated python 3.6, add support for python 3.10 #107

merged 2 commits into from
Aug 1, 2022

Conversation

istr
Copy link
Contributor

@istr istr commented Jul 23, 2022

Added support for python 3.10 (changed to collections.abc.Iterable) with fallback for legacy 2.7.
Dropped 3.6 since it reached EOL as per https://devguide.python.org/versions/

@ross FYI - this affects octodns_ns1 when using python 3.10.

@istr
Copy link
Contributor Author

istr commented Jul 23, 2022

This addresses the compatibility issue with 2.7 mentioned in #75 as well, the local tox tests passed.

ross
ross previously approved these changes Jul 23, 2022
@istr
Copy link
Contributor Author

istr commented Jul 23, 2022

@ross Thanks for your approval. However, it seems that a maintainer needs to approve to trigger the tests for me, since this is my first contribution to this repo.

@ross
Copy link

ross commented Jul 23, 2022

@ross Thanks for your approval. However, it seems that a maintainer needs to approve to trigger the tests for me, since this is my first contribution to this repo.

Yeah, I was just signaling that it makes sense to me. 😁

@cbertinato
Copy link
Contributor

Going to force this through because the lint failure is not the fault of this PR.

@cbertinato cbertinato merged commit 831dcd3 into ns1:master Aug 1, 2022
@brianeclow brianeclow mentioned this pull request Aug 15, 2022
@istr istr deleted the python-3.10-support branch August 16, 2022 17:06
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.

3 participants