Skip to content

Commit 2b23046

Browse files
committed
Stop trying to upload the coverage if there is no auth token
This fixes build failures on pull requests from forks (where usually no authentication token for the code coverage is set). Also fix the name of the coverage upload step to contain the corect name of the service.
1 parent 2a2a167 commit 2b23046

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/ci.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,9 +72,9 @@ jobs:
7272
- name: Run Tests
7373
run: ./vendor/bin/phpunit --coverage-clover build/coverage/xml
7474

75-
- name: Upload coverage results to Coveralls
76-
if: "${{ matrix.coverage != 'none' }}"
75+
- name: Upload coverage results to Codacy
7776
env:
7877
CODACY_PROJECT_TOKEN: ${{ secrets.CODACY_PROJECT_TOKEN }}
78+
if: "${{ matrix.coverage != 'none' && env.CODACY_PROJECT_TOKEN != '' }}"
7979
run: |
8080
./vendor/bin/codacycoverage clover build/coverage/xml

0 commit comments

Comments
 (0)