search

liuggio / fastest

Simple parallel testing execution... with some goodies for functional tests.
MIT License
473 stars 65 forks source link

Update travis.yml with PHP 7.1 #93

Closed DonCallisto closed 7 years ago

DonCallisto commented 7 years ago

Updated travis.yml to test codebase against PHP 7.1

DonCallisto commented 7 years ago

There's an error with PHP 7.1

Would you like me to tackle it directly in this PR?

alexislefebvre commented 7 years ago

We can't merge a PR that break the test suite so yes you'll probably have to fix the issue in this PR.

I didn't know that we can configure dist per job, that's nice!

DonCallisto commented 7 years ago

@alexislefebvre I know that's not correct to merge a PR with red tests, I was asking only for the right way to handle this. So, I'm going to take a look to those errors directly in this PR

DonCallisto commented 7 years ago

@alexislefebvre the build is now green 🎉

liuggio commented 7 years ago

@DonCallisto thanks a lot

DonCallisto commented 7 years ago

You are very welcome! 😄