Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Ensure Travis fails when tests fail [ignore CI failure] #106

Merged
merged 4 commits into from
Oct 27, 2018

Conversation

ehashman
Copy link
Member

This should fix #105.

@ehashman
Copy link
Member Author

Hmm, I want the tests to fail, not error. Guess I'll try something else...

@ehashman
Copy link
Member Author

So the current test failure is legitimate thanks to #107. I'm going to try to fix that too, but feel free to merge this PR and I can submit that separately.

@ehashman ehashman changed the title Ensure Travis fails when tests fail Ensure Travis fails when tests fail [ignore CI failure] Oct 18, 2018
@njsmith
Copy link
Member

njsmith commented Oct 18, 2018

I think we should replace the script block by an actual script file, that uses set -euxo pipefail.

@ehashman
Copy link
Member Author

-u complains that $WHEELHOUSE is undefined so I left that out.

@njsmith
Copy link
Member

njsmith commented Oct 20, 2018

+1

@ehashman
Copy link
Member Author

This should be good to go!

@ehashman ehashman requested a review from njsmith October 21, 2018 20:12
@safijari
Copy link

bump

@ehashman
Copy link
Member Author

I got a +1 so once CI completes for this merge of the new changes I will merge.

@ehashman ehashman merged commit f270c41 into pypa:master Oct 27, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Failing tests in Travis have incorrect exit status
3 participants