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

Comments

jonathan1055 created an issue. See original summary.

jonathan1055’s picture

Here is the log from 4th Jan which ran OK
travis log OK

and here is the failed log of 11th Jan
travis log failed

jonathan1055’s picture

Status: Active » Needs review
StatusFileSize
new549 bytes

Here 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

  • fago committed 5cae367 on 8.x-3.x authored by jonathan1055
    Issue #2937362 by jonathan1055: Fix composer install syntax in .travis....
fago’s picture

Status: Needs review » Fixed

thx, merged.

jonathan1055’s picture

That'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

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.