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

Use tox to run tests on Travis #7

Merged
merged 1 commit into from
May 22, 2015
Merged

Use tox to run tests on Travis #7

merged 1 commit into from
May 22, 2015

Conversation

ryanhiebert
Copy link
Contributor

tox-travis helps make the configuration simple, running all of the tests for a particular python version on the Travis matrix version of Python.

Add tests of Python 3.3, and drop Python 3.4 from testing with Django 1.6, since it does not support that version of Python.

tox-travis helps make the configuration simple, running all of the tests
for a particular python version on the Travis matrix version of Python.

Add tests of Python 3.3, and drop Python 3.4 from testing with Django
1.6, since it does not support that version of Python.
sheppard added a commit that referenced this pull request May 22, 2015
Use tox to run tests on Travis
@sheppard sheppard merged commit 225d62e into openwisp:master May 22, 2015
@sheppard
Copy link
Collaborator

Thanks, this is useful. I've been thinking about using tox more but haven't had time to look into it.

For this small project, what you put together works very well. If it was a much larger project (like DRF), I might want to have a separate Travis build for each tox environment. Even though it'd be a bit less efficient, it would make it easier to pinpoint exactly which combination of versions are failing.

@ryanhiebert
Copy link
Contributor Author

Yep. It's a trade-off. tox-travis is good for the simple cases, but there are places where you just have to work around the limitations of Travis' build matrix.

@ryanhiebert ryanhiebert deleted the tox branch May 22, 2015 15:30
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