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

Travis: Set exit to true in nose.main #4708

Merged
merged 1 commit into from
Jul 16, 2015

Conversation

jenshnielsen
Copy link
Member

Following the merge of #4668 tests do not fail correctly on Travis. This sets exit to True in nose.main. Otherwise the exit status is not set correctly and Travis does not fail on failure.

Compare https://travis-ci.org/matplotlib/matplotlib/jobs/71108096 which should have failed but does not to the result of the python 3.5 job on this branch

Otherwise the exit status is not set correctly and travis does not fail
@jenshnielsen
Copy link
Member Author

@matplotlib/developers Please be careful when merging PR's until this is fixed. The Travis status will be set to passed incorrectly. Im sorry that I did not notice before

tacaswell added a commit that referenced this pull request Jul 16, 2015
Travis: Set exit to true in nose.main
@tacaswell tacaswell merged commit fe5a536 into matplotlib:master Jul 16, 2015
@tacaswell
Copy link
Member

3.5 fails, so this looks good 👍

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.

None yet

2 participants