Update travis.yml with PHP 7.1

8p / EightPointsGuzzleBundle

⛽️ Integrates Guzzle 6.x, a PHP HTTP Client, into Symfony

MIT License

440 stars 71 forks source link

Update travis.yml with PHP 7.1 #108

Closed DonCallisto closed 7 years ago

DonCallisto commented 7 years ago

As PHP 7.1 is out since December (6 months) I think is useful to test the codebase against it.

[ ] Bug fix
[ ] New feature
[ ] BC breaks
[ ] Deprecations
[x] Tests pass

License: MIT

DonCallisto commented 7 years ago

I didn't find a way to allow_failure for hhvm and symfony-master. Suggestions are welcome (keeping in mind that hhvm support will be dropped in Symfony 4)

florianpreusner commented 7 years ago

You are absolutely right! Thanks a lot!

DonCallisto commented 7 years ago

@florianpreusner let me know if travis.yml could be improved somehow, I'm not that expert but I'm willing to improve the configuration.

florianpreusner commented 7 years ago

Well, I think I'm going to remove hhvm in the future because of php 7. I don't see any need to still support/test hhvm. But this is a bigger change for some users, that's why it should be part of a new version.