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

run tox against multiple python versions #142

Merged
merged 5 commits into from
Aug 24, 2016

Conversation

Rob-Johnson
Copy link
Contributor

@Rob-Johnson Rob-Johnson commented Aug 22, 2016

I'm not sure this'll work...but it seems to locally?

Fixes #80.

@Rob-Johnson
Copy link
Contributor Author

I think the failures here are all legitimate, and there are also a few hidden:

  • something has changed with the mocking in python3?
  • the itests are failing (see the syntax errors when running with python3), but exiting with success

I'll keep iterating until it's fixed

@Rob-Johnson
Copy link
Contributor Author

@solarkennedy I think this is good to go?

@@ -1,44 +1,28 @@
[tox]
passenv = TRAVIS
usedevelop=True
basepython = python2.7
envlist = py,pep8
envlist={test,itest}-{py27,py33}
Copy link
Contributor

Choose a reason for hiding this comment

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

Is pep8 lost or is it run through a different means?

@solarkennedy solarkennedy merged commit e6f2f3d into thefactory:master Aug 24, 2016
@solarkennedy
Copy link
Contributor

Thanks!

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