Skip to content

Travis: use "set -e" to exit script in case of errors #890

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

Merged
merged 1 commit into from
Nov 25, 2017

Conversation

blueyed
Copy link
Contributor

@blueyed blueyed commented Nov 12, 2017

blueyed referenced this pull request Nov 12, 2017
* Improve Travis / tox setup

Mainly inspired by using pytest directly in tox, for easy test selection
etc.

* Travis: use lightweight containers (sudo=false)

* coverage/codecov integration

* include TEST_EXTRA as codecov flag

* codecov: -X fix

This hopefully works around `complex/complex/commands/__init__.py` and
`termui/termui.py` showing up in `examples` (at the root).
@stefreak
Copy link
Contributor

I cherry-picked this into #889 and now it fails as it's supposed to be :) Seems to be working. 👍

@stefreak
Copy link
Contributor

@mitsuhiko this really should be merged, because right now the tests are always green, even if they actually fail. According to the contributors page, you did the most commits, that's why I'm mentioning you.

@blueyed
Copy link
Contributor Author

blueyed commented Nov 25, 2017

Ping @untitaker.

@mitsuhiko mitsuhiko merged commit 2ab7a2b into pallets:master Nov 25, 2017
@mitsuhiko
Copy link
Contributor

Indeed. Agreed.

@blueyed blueyed deleted the fix-travis branch November 25, 2017 22:56
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants