Skip to content

Travis: Fix tools testing breakage due to multiple python versions #6200

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

Closed
wants to merge 7 commits into from

Conversation

geky
Copy link
Contributor

@geky geky commented Feb 23, 2018

Travis doesn't recognize version matrices when inside a matrix include job. Instead, it breaks in a way that isn't reported as an error.

Fix is to just manually run the python versions we want to test.

Note: This will result in failures, as it already has for #6195. Will fix as failures are reported.

Please ignore the commit noise for now, will squash at the end.

introduced in #5848
intended for patch
cc @deepikabhavnani, @cmonr, @theotherjimmy
related travis-ci/travis-ci#9275

Travis doesn't recognize version matrices when inside a matrix include job.
Instead, it breaks in a way that isn't reported as an error.

Fix is to just manually run the python versions we want to test.
@cmonr cmonr requested a review from theotherjimmy February 23, 2018 21:58
@geky
Copy link
Contributor Author

geky commented Feb 26, 2018

Python 3 support in Ubuntu is being a pain. Here's a temporary fix that just removes python 3 testing so we can at least run some tool tests on master: #6218

@deepikabhavnani deepikabhavnani removed their request for review February 27, 2018 05:40
@geky
Copy link
Contributor Author

geky commented Mar 5, 2018

Handing off to @cmonr

@geky geky closed this Mar 5, 2018
@geky geky removed the needs: work label Mar 5, 2018
@geky geky deleted the g-fix-travis-python-versions branch April 20, 2018 16:26
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