-
Notifications
You must be signed in to change notification settings - Fork 16
Travis version support #21
Copy link
Copy link
Closed
Labels
bugSomething isn't working. It's clear that this does need to be fixed.Something isn't working. It's clear that this does need to be fixed.enhancementNew feature or improved functionality.New feature or improved functionality.
Metadata
Metadata
Assignees
Labels
bugSomething isn't working. It's clear that this does need to be fixed.Something isn't working. It's clear that this does need to be fixed.enhancementNew feature or improved functionality.New feature or improved functionality.
@bladeSk as you have added PHP 7 support, I think it's important that our test suite runs against multiple versions. If you look at https://github.com/ably/ably-php/blob/master/.travis.yml#L3, you will see we only support 5.4 now. Should we not run the test suite against all support PHP versions to make sure we don't introduce regressions in future?
See https://docs.travis-ci.com/user/languages/php/