Skip to content

Conversation

@mwcraig
Copy link
Contributor

@mwcraig mwcraig commented Nov 28, 2021

This PR does a few things:

  • Drops tests on Python 3.7 and adds tests on Python 3.10. The minimum Python version is still 3.7.
  • Build wheels on Python 3.7 through 3.10
  • Drop a version check from setup.py that adds autobahn as a dependency for some old Python versions. We now require autobahn always.
  • Remove logic from __init__.py that is trying to check for Python 2.7 or 3.4, neither of which we support.

This does NOT mean Python 3.7 is no longer supported. Just trying to keep
the number of builds a little reasonable.
@mwcraig mwcraig marked this pull request as draft November 28, 2021 20:27
@mwcraig mwcraig marked this pull request as ready for review November 28, 2021 20:44
matrix:
python-version: [3.7, 3.8, 3.9]
platform: [macos-latest, windows-latest] # No wheels on linux yet
python-version: [3.7, 3.8, 3.9, 3.10]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this 3.10 be '3.10' like the others? I see it's in a '.yml' file, so maybe it's smart enough, but if these get parsed as a python list, it might end up as 3.1 rather than '3.10'.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Weird -- I had already changed that...not sure why the quotes aren't showing up for you.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you see the quotes there now? Line 14 of .github/workflows/upload_pypi.yml ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed! I see them in your branch, but not in the PR for some reason. Weird.

@mwcraig mwcraig merged commit 5fc722c into vpython:master Jan 7, 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.

2 participants