Use PHPCS/Coder 8.3.3 in d.o. testing and travis builds

Trial patch for drupalci.yml

That worked! Now to really test it with a modified file which should have some coding standards faults.

PHPCS was still 8.3.2 - it was not reloaded, I guess the patch to drupal's composer.json was executed too late to be effective.
Try another way, by specifying the coder version directly in our drupalci.yml

That did not make any difference. Try modifying composer.json

yes, that same old problem. But at least PHPCS worked this time.

The travis tests now pass at core 8.8
https://travis-ci.org/jonathan1055/scheduler/builds/534136964

The patch https://www.drupal.org/files/issues/2019-04-22/3049433-8.patch is applied to core composer.json and composer.lock after cloning drupal but before running composer self-update and install.

Patch without changes to drupalci.yml just composer. Checking syntax ">=8.3.3"

That's good, all as expected. The initial version loaded is:

Installing drupal/coder (8.3.2): Cloning 44c80c2107

But after the patch is applied we get:

- Updating drupal/coder (8.3.2 => 8.3.3):  Checking out a33d3388fb

Committed before the re-test had run, so patch failed to apply, but all OK.

Good. We now have PHPCS running properly on our contrib patches at core 8.8.x in both drupal testing and travis build testing.