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

Un-hardcode the supported factors again #1377

Merged
merged 1 commit into from Jul 27, 2019
Merged

Un-hardcode the supported factors again #1377

merged 1 commit into from Jul 27, 2019

Conversation

asottile
Copy link
Contributor

Resolves #1374

This also fixes some other things which regressed in #983 -- notably this

$ tox -e py21 --notest && .tox/py21/bin/python --version
py21 create: /tmp/tox/.tox/py21
py21 installdeps: pip == 19.1.1
py21 inst: /tmp/tox/.tox/.tmp/package/1/tox-3.13.3.dev7+g84c4c8d.tar.gz
py21 installed: apipkg==1.5,atomicwrites==1.3.0,attrs==19.1.0,coverage==4.5.3,execnet==1.6.1,filelock==3.0.12,flaky==3.6.0,freezegun==0.3.12,importlib-metadata==0.18,more-itertools==7.2.0,packaging==19.0,pathlib2==2.3.4,pluggy==0.12.0,psutil==5.6.3,py==1.8.0,pyparsing==2.4.1.1,pytest==5.0.1,pytest-cov==2.7.1,pytest-forked==1.0.2,pytest-mock==1.10.4,pytest-randomly==1.2.3,pytest-xdist==1.29.0,python-dateutil==2.8.0,six==1.12.0,toml==0.10.0,tox==3.13.3.dev7+g84c4c8d,virtualenv==16.7.1,wcwidth==0.1.7,zipp==0.5.2
_______________________________________________________________________ summary ________________________________________________________________________
  py21: skipped tests
  congratulations :)
Python 3.6.8

restored behaviour:

$ .tox/py36/bin/tox -e py21 --notest
py21 create: /tmp/tox/.tox/py21
SKIPPED: InterpreterNotFound: python2.1
_______________________________________________________________________ summary ________________________________________________________________________
SKIPPED:  py21: InterpreterNotFound: python2.1
  congratulations :)

@todo
Copy link

todo bot commented Jul 25, 2019

(stephenfin): Raise an exception here in tox 4.0

# TODO(stephenfin): Raise an exception here in tox 4.0
warnings.warn(
"conflicting basepython version (set {}, should be {}) for env '{}';"
"resolve conflict or set ignore_basepython_conflict".format(
proposed_version, implied_version, testenv_config.envname
)


This comment was generated by todo based on a TODO comment in 84c4c8d in #1377. cc @asottile.

@gaborbernat
Copy link
Member

Looks good 👍 if the ci passes 👍

@asottile
Copy link
Contributor Author

the py34 failures appear to be due to #1375

@gaborbernat
Copy link
Member

Can you fix the merge conflict? How we fix the py34 thing until upstream fixes it?

@asottile
Copy link
Contributor Author

I think we can pyparsing!=2.4.1,!=2.4.1.1 to work around the bug -- let me make a PR and see if that's doable

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

What to do about python3.10
2 participants