Skip to content

Commit 8f3477b

Browse files
committed
WIP Remove Scrutinizer, Add Coveralls
Scrutinizer has been a source of problems for a long time. False positives, unavailability, not open source, irrelevant complexity flags. I am a bit concerned because Scrutinizer sometimes seems to run even without a yaml file. We shall see. I do still want coverage reports. Instead of uploading them to Scrutinizer, I will try Coveralls, which is used by PhpWord.
1 parent 63d3960 commit 8f3477b

File tree

2 files changed

+9
-33
lines changed

2 files changed

+9
-33
lines changed

.github/workflows/main.yml

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -250,9 +250,15 @@ jobs:
250250

251251
- name: Coverage
252252
run: |
253-
./vendor/bin/phpunit --coverage-clover coverage-clover.xml
254-
composer global require scrutinizer/ocular
255-
~/.composer/vendor/bin/ocular code-coverage:upload --format=php-clover coverage-clover.xml
253+
./vendor/bin/phpunit --coverage-clover build/clover.xml
254+
255+
- name: Upload coverage results to Coveralls
256+
env:
257+
COVERALLS_REPO_TOKEN: ${{ secrets.GITHUB_TOKEN }}
258+
run: |
259+
wget https://github.com/php-coveralls/php-coveralls/releases/download/v2.4.3/php-coveralls.phar
260+
chmod +x php-coveralls.phar
261+
php php-coveralls.phar --coverage_clover=build/clover.xml --json_path=build/coveralls-upload.json -vvv
256262
257263
release:
258264
permissions:

.scrutinizer.yml

Lines changed: 0 additions & 30 deletions
This file was deleted.

0 commit comments

Comments
 (0)