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

Support Python 3.10 #186

Merged
merged 2 commits into from
Mar 14, 2022
Merged

Conversation

PeterJCLaw
Copy link
Contributor

@PeterJCLaw PeterJCLaw commented Mar 11, 2022

Fixes #184

This moves away from unittest2 and then claims support for Python 3.10 (which otherwise appears to already work fine).

unittest2 appears not to be compatible with Python 3.101, so this is a stepping stone towards getting testing working there.

unittest2 also appears to be somewhat unmaintained (based on the activity on https://github.com/testing-cabal/unittest-ext), which is somewhat unsurprising given that Python 2 is no longer maintained.

In any case, the built-in unittest module seems to have all that's needed here.

Footnotes

  1. specifically it relies on various types which have moved out of collections.

@PeterJCLaw

This comment was marked as outdated.

@PeterJCLaw PeterJCLaw marked this pull request as ready for review March 11, 2022 18:58
@PeterJCLaw PeterJCLaw marked this pull request as draft March 11, 2022 21:00
@PeterJCLaw PeterJCLaw marked this pull request as ready for review March 11, 2022 21:00
@PeterJCLaw
Copy link
Contributor Author

Hrm, not sure why Travis isn't running on this PR?

@dannyroberts
Copy link
Member

Yeah I was having the same issue. If you build this PR on top of #190 (or wait for it to get merged), that'll fix any issues with travis (by eliminating travis). If you do that, I suggest you also add python 3.10 to the matrix, since I think that's the point of this PR. Thanks again for the contributions @PeterJCLaw!

unittest2 appears not to be compatible with Python 3.10, so this
is a stepping stone towards getting testing working there.
@PeterJCLaw PeterJCLaw changed the base branch from master to github-actions March 11, 2022 21:48
@PeterJCLaw PeterJCLaw changed the title Move to using builtin unittest for testing Support Python 3.10 Mar 11, 2022
This seems to already work, so this just enables testing and adds
a classifier.
@dannyroberts dannyroberts merged commit 347769e into dimagi:master Mar 14, 2022
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.

unable to compile jsonobject-0.9.10 for python 3.10
2 participants