Skip to content

Commit 3bce4d3

Browse files
committed
CI: switched Coveralls upload to the official action
The php-coveralls phar is replaced by coverallsapp/github-action, which resolves the token and the absolute paths in the Clover report on its own, so tests/.coveralls.yml is no longer needed. Coverage is now uploaded even when tests crash under phpdbg. Being phpdbg-compatible is not a goal, but until now the failing step aborted the job before the upload ever ran. The job-level continue-on-error is gone, so genuine failures, including a rejected upload, are visible.
1 parent 2d88aa5 commit 3bce4d3

2 files changed

Lines changed: 5 additions & 9 deletions

File tree

‎.github/workflows/tests.yml‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,6 @@ jobs:
4545
code_coverage:
4646
name: Code Coverage
4747
runs-on: ubuntu-latest
48-
continue-on-error: true
4948
steps:
5049
- uses: actions/checkout@v6
5150
- uses: shivammathur/setup-php@v2
@@ -55,10 +54,11 @@ jobs:
5554

5655
- run: composer install --no-progress --prefer-dist
5756
- run: composer tester -- -p phpdbg --coverage ./coverage.xml --coverage-src ./src
58-
- run: wget https://github.com/php-coveralls/php-coveralls/releases/download/v2.4.3/php-coveralls.phar
59-
- env:
60-
COVERALLS_REPO_TOKEN: ${{ secrets.GITHUB_TOKEN }}
61-
run: php php-coveralls.phar --verbose --config tests/.coveralls.yml
57+
continue-on-error: true
58+
- uses: coverallsapp/github-action@v2
59+
with:
60+
file: coverage.xml
61+
format: clover
6262

6363

6464
jasmine:

‎tests/.coveralls.yml‎

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

0 commit comments

Comments
 (0)