Update .travis.yml

[Klizzy]

removed PHP-7.4 from .travis.yml allow_failures to make it compatible with PHP 7.4.
See #1023

[greg0ire]

Hi! If we no longer allow 7.4 to fail, we also need to fix the failure described here: https://travis-ci.org/doctrine/DoctrineBundle/jobs/591925826

[nimah79]

This problem is related to Twig, not Doctrine. So, this PR can be merged temporarily.

[greg0ire]

It really shouldn't IMO. If we merge it, builds that were previously green will become red for no good reason. If something has to be done temporarily, it's listing that 7.4 job as part of allowed_failures, and that's what we already do.

[xabbuh]

What about using PHP 7.4 for the build that uses not yet released dependencies versions? That should work as Twig already has a patch for this just without a stable release yet, shouldn't it?

[greg0ire]

I think it would make a lot of sense!

@nimah79 here is what I think @xabbuh is talking about:

DoctrineBundle/.travis.yml

Lines 49 to 55 in af8ac79

	# Test against latest Symfony 4.4 dev
	- php: 7.3
	env: SYMFONY_VERSION="4.4.*@dev" SYMFONY_DEPRECATIONS_HELPER=1
	install:
	- composer config minimum-stability dev
	- composer require --dev "symfony/messenger:4.4.*@dev" --no-update
	- travis_retry composer update -n --prefer-dist

[fabpot]

@greg0ire I've just released Twig 2.12 which should unlock this PR.

[fabpot]

And tests are green now!

[SenseException]

Thank you @fabpot 👍

[greg0ire]

Thanks @fabpot! Merged!

[greg0ire]

I think I merged a bit too fast… maybe we should require the new version? If yes, see #1025