Add Scrutinizer CI#206
Conversation
Majkl578
force-pushed
the
scrutinizer
branch
2 times, most recently
from
7abfec2 to
05e3917
Compare
|
Didn't find any way to disable phpcs run (and Patches) so let's just ignore it until we add CS later. |
| - | | ||
| if [ "x$COVERAGE" == "xyes" ] && [[ ! $(php -m | grep -si xdebug) ]]; then | ||
| echo "xdebug is required for coverage" | ||
| exit 1 |
There was a problem hiding this comment.
using exit in a command put in the .travis.yml will exit the whole build script, which will break the Travis output (as reporting for this command will not be done, as well as following tasks).
There was a problem hiding this comment.
First time I hear about it. 🤔 What would you suggest as replacement here?
There was a problem hiding this comment.
Well, there used to be a warning in the Travis doc about it in the past, but I cannot find it anymore (they reorganized their doc in the meantime).
I thinkk you can use travis_terminate 1 for that (which is a command which will exit with 1 as exit code, and so become a failure).
3 participants
Basically copied from Migrations and CS condition removed (not used in this repo, yet) and added coverage.