Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Travis: Use stable versions of Coveralls #550

Merged
merged 3 commits into from
Dec 24, 2017

Conversation

jrfnl
Copy link
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
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
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
@jrfnl jrfnl added the chores/QA label Mar 8, 2018
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