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

Python 3.8 support #151

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

willfurnass
Copy link
Contributor

@willfurnass willfurnass commented Feb 10, 2020

Python 3.8 support required PR #150 to be merged, which has now been done.

Also ensures that all tests run with Python 3.6, 3.7 and 3.8 but coverage only calculated and uploaded for Python 3.7 (could not do this for just 3.8 instead as coverage doesn't work with Python 3.8 yet - see nedbat/coveragepy#772).

@coveralls
Copy link

Coverage Status

Coverage remained the same at 11.314% when pulling a26d650 on willfurnass:py38 into 2c8bd97 on robclewley:master.

@willfurnass
Copy link
Contributor Author

Hmm, odd that Travis CI jobs now only fail when Python=3.7 and COVERAGE="yes". NB those tests result in multiple occurrences of:

...
           if self._parameterized:
>               self.indepvararray.resize(new_len)
E               ValueError: cannot resize an array that references or is referenced
E               by another array in this way.  Use the resize function
...

@robclewley robclewley mentioned this pull request Jan 2, 2021
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