Skip to content

Add Python 3.8 tests#851

Merged
corranwebster merged 6 commits into
mainfrom
fix/python-3.8-ci
Jan 24, 2023
Merged

Add Python 3.8 tests#851
corranwebster merged 6 commits into
mainfrom
fix/python-3.8-ci

Conversation

@corranwebster
Copy link
Copy Markdown
Contributor

@corranwebster corranwebster commented Jan 23, 2023

This adds a parallel set of tests for Python 3.8 rather than trying to adapt the current set of tests. Also updates edmtool to allow Python 3.8 and PySide6, and pyproject.toml to allow for known issue with setuptools.

Needed to make one change to code due to newer version of NumPy being picked up.

No requirement that they pass for PRs to be mergeable yet, but will likely add that once this PR is approved.

@corranwebster corranwebster merged commit 23c4fd9 into main Jan 24, 2023
@corranwebster corranwebster deleted the fix/python-3.8-ci branch January 24, 2023 07:55
corranwebster added a commit that referenced this pull request Jan 24, 2023
This adds a parallel set of tests for Python 3.8 rather than trying to adapt the current set of tests.  Also updates edmtool to allow Python 3.8 and PySide6, and pyproject.toml to allow for known issue with setuptools.

Needed to make one change to code due to newer version of NumPy being picked up.

No requirement that they pass for PRs to be mergeable yet, but will likely add that once this PR is approved.
corranwebster added a commit that referenced this pull request Jan 24, 2023
This adds a parallel set of tests for Python 3.8 rather than trying to adapt the current set of tests.  Also updates edmtool to allow Python 3.8 and PySide6, and pyproject.toml to allow for known issue with setuptools.

Needed to make one change to code due to newer version of NumPy being picked up.

No requirement that they pass for PRs to be mergeable yet, but will likely add that once this PR is approved.
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