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

Restore testing on Python 2, bypassing the requires-python check when installing Setuptools. #1961

Merged
merged 2 commits into from
Jan 12, 2020

Conversation

jaraco
Copy link
Member

@jaraco jaraco commented Jan 12, 2020

Follow-up to #1955, this change allows the tests to run on Python 2 even though the packages declares a requirement on Python 3.5 or later. This change will prevent the risk of bit-rot on the master and maintain the expectation that Python 2 compatibility is retained for those installers not honoring PEP 345.

@jaraco
Copy link
Member Author

jaraco commented Jan 12, 2020

This technique worked for me locally. I'm not yet sure why it's not working here.

@jaraco jaraco changed the title Restore testing on Python 2, bypassing the requires-python check when installing Setuptools. [WIP] Restore testing on Python 2, bypassing the requires-python check when installing Setuptools. Jan 12, 2020
@jaraco jaraco force-pushed the restore-python2-tests branch from 800b20c to 04ef944 Compare January 12, 2020 16:44
@jaraco jaraco force-pushed the restore-python2-tests branch from 04ef944 to 8e6b993 Compare January 12, 2020 16:44
@jaraco
Copy link
Member Author

jaraco commented Jan 12, 2020

aha. I know why it's failing - because I'm invoking tox with tox -e py27, but travis is using tox -e python (on Python 2.7).

@jaraco jaraco force-pushed the restore-python2-tests branch from d2c4272 to cb2138e Compare January 12, 2020 17:43
@jaraco jaraco changed the title [WIP] Restore testing on Python 2, bypassing the requires-python check when installing Setuptools. Restore testing on Python 2, bypassing the requires-python check when installing Setuptools. Jan 12, 2020
@jaraco jaraco merged commit 7bac1ed into master Jan 12, 2020
@jaraco jaraco deleted the restore-python2-tests branch January 12, 2020 18:16
webknjaz added a commit to webknjaz/setuptools that referenced this pull request Jan 14, 2020
webknjaz added a commit to webknjaz/setuptools that referenced this pull request Jan 14, 2020
As per pypa#1961.

P.S. Using `contains(['2.7', 'pypy2'], matrix.python-version)` does not
work even though
https://help.github.com/en/actions/automating-your-workflow-with-github-actions/contexts-and-expression-syntax-for-github-actions#contains
promises arrays to be supported.
webknjaz added a commit to webknjaz/setuptools that referenced this pull request Jan 14, 2020
As per pypa#1961.

P.S. Using `contains(['2.7', 'pypy2'], matrix.python-version)` does not
work even though
https://help.github.com/en/actions/automating-your-workflow-with-github-actions/contexts-and-expression-syntax-for-github-actions#contains
promises arrays to be supported.
webknjaz added a commit to webknjaz/setuptools that referenced this pull request Jan 20, 2020
As per pypa#1961.

P.S. Using `contains(['2.7', 'pypy2'], matrix.python-version)` does not
work even though
https://help.github.com/en/actions/automating-your-workflow-with-github-actions/contexts-and-expression-syntax-for-github-actions#contains
promises arrays to be supported.
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.

1 participant