Skip to content

Travis: Use stable versions of Coveralls#550

Merged
wimg merged 3 commits intomasterfrom
feature/update-coveralls-config
Dec 24, 2017
Merged

Travis: Use stable versions of Coveralls#550
wimg merged 3 commits intomasterfrom
feature/update-coveralls-config

Conversation

@jrfnl
Copy link
Copy Markdown
Member

@jrfnl jrfnl commented Dec 23, 2017

The PHP-Coveralls project has finally tagged some releases. Using the tagged versions allows Travis to cache the package download which should hopefully make the builds a little faster.

The minimum PHP version for v 2.x is PHP 5.5, for the 1.x branch, it's PHP 5.3.3, so depending on the build, different versions are being used.

Other notes:

  • The repository has moved from satooshi/php-coveralls to php-coveralls/php-coveralls.
  • As of PHPCoveralls 2.x, the command to call coveralls has changed from coveralls to php-coveralls

This PR takes all of the above into account. Keeping my fingers crossed that the build will pass & coverage will be reported correctly (again).

Fixes #549 (hopefully... let's see if it really did after the build has run)

The PHP-Coveralls project has *finally* tagged some releases. Using the tagged versions allows Travis to cache the package download which should hopefully make the builds a little faster.

The minimum PHP version for v 2.x is PHP 5.5, for the 1.x branch, it's PHP 5.3.3, so depending on the build, different versions are being used.

Other notes:
* The repository has moved from `satooshi/php-coveralls` to `php-coveralls/php-coveralls`.
* As of PHPCoveralls 2.x, the command to call coveralls has changed from `coveralls` to `php-coveralls`

This PR takes all of the above into account. Keeping my fingers crossed that the build will pass & coverage will be reported correctly (again).

Fixes 549
@jrfnl jrfnl added this to the 8.1.0 milestone Dec 23, 2017
@jrfnl jrfnl requested a review from wimg December 23, 2017 01:22
Coveralls is run against the highest and the lowest PHP version to get the best results. As PHP 7.2 has been released, the highest PHP version is now 7.2
@jrfnl
Copy link
Copy Markdown
Member Author

jrfnl commented Dec 23, 2017

Ok, just checked the Travis logs, but apparently it's still not working.
Looks like the bash file needed isn't copied to the vendor/bin directory. Will see what I can do.

@jrfnl
Copy link
Copy Markdown
Member Author

jrfnl commented Dec 23, 2017

Found it & fixed it!

@wimg wimg merged commit a3d68fd into master Dec 24, 2017
@wimg wimg deleted the feature/update-coveralls-config branch December 24, 2017 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants