Skip to content

Commit

Permalink
dev: Coveralls changed the name of the executable
Browse files Browse the repository at this point in the history
We use coveralls.io to measure the code coverage over time.
With some release the executable has been renamed so it
didn't uploads coverage.xml anymore in the end.
To prevent this and other possible issues
we install coverals alone and run the executable for '^2.2'.

* .gitignore: Also ignore the coverage file in root dir
  • Loading branch information
rmp-up authored and pretzlaw committed Dec 15, 2019
1 parent 57fc5eb commit 455d9bb
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
/vendor

/.coveralls.yml
/coverage.xml
.docker
.env
composer.lock
Expand Down
5 changes: 3 additions & 2 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -84,8 +84,9 @@ script:

after_script:
# upload coverage.xml file to Coveralls to analyze it
- composer require php-coveralls/php-coveralls # reinstall dev
- vendor/bin/coveralls --verbose
- rm composer.json composer.lock # to prevent conflicts
- composer require --dev 'php-coveralls/php-coveralls:^2.2'
- travis_retry php vendor/bin/php-coveralls

after_failure:
- cat srv/wp-content/debug.log || true

0 comments on commit 455d9bb

Please sign in to comment.