The .travis.yml build script has the following line:
# Run composer install in the module directory in order to fetch dev
# dependencies there. This includes phpcs & coder for checking coding style.
- travis_retry composer -d=$TESTDIR install
This has been working perfectly well up to Jan 4th. I have a Travis cron task which runs weekly on the default 8.x-3.x branch, and the following run (on Jan 11th) failed, with the problem being that PHPCS was not installed, so the coding standards checks failed. The output from the attempted install implied that the syntax could be wrong. The first thing I noticed was that the help which was shown implied that no "=" was needed when using the -d parameter. Secondly it reads a bit odd to have the parameter $TESTDIR before the actual command install. After some trial and error I found that both of these things are now classed as incorrect (must be some tightening up of validation in Composer). The "=" sign has to be removed and also the "install" has to come before the parameters. So what we need is
- travis_retry composer install -d $TESTDIR
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 2937362-travis-build.patch | 549 bytes | jonathan1055 |
| #2 | 2018-01-11 composer install failed.png | 495.52 KB | jonathan1055 |
| #2 | 2018-01-04 composer install OK.png | 249.16 KB | jonathan1055 |
Comments
Comment #2
jonathan1055 commentedHere is the log from 4th Jan which ran OK

and here is the failed log of 11th Jan

Comment #3
jonathan1055 commentedHere is the commit in my github repo.
https://github.com/jonathan1055/rules/commit/e7b78c39fe15a73e74342628116...
Here is the Travis build which shows it is working with this change
https://travis-ci.org/jonathan1055/rules/builds/329970744
Attached is a patch for info (and for anyone's own testing on Travis) but the change has no effect on drupal.org testbots.
There might be further enhancements to make to this, such as dropping the 8.3.x build and adding 8.6.x so I will create a pull-request later.
Jonathan
Comment #5
fagothx, merged.
Comment #6
jonathan1055 commentedThat's great. Now at least we have a default branch which should start running again in Travis. I will raise another issue to discuss when to drop 8.3.x and start using 8.6.x