Skip to content

Commit

Permalink
Got code coverage working again via XDebug, updated CI to enforce cod…
Browse files Browse the repository at this point in the history
…… (#72)

* Got code coverage working again via XDebug, updated CI to enforce code coverage

* Attempting to switch to main php-coveralls repo to publish coverage reports

* Changed to using php-coveralls rather than the fork for the rest of the Aphiria libraries now that it seems to be working
  • Loading branch information
davidbyoung committed Oct 17, 2020
1 parent d5b33d9 commit 4fdb423
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,19 +20,19 @@ jobs:
uses: shivammathur/setup-php@v2
with:
php-version: ${{ matrix.php }}
extensions: curl, dom, libxml, mbstring, pcntl, zip
extensions: curl, dom, libxml, mbstring, pcntl, xdebug-beta, zip
tools: composer:v2
coverage: pcov
coverage: xdebug
- name: Install Dependencies
run: composer update --${{ matrix.stability }} --prefer-dist --no-interaction --no-progress
- name: Run Tests
run: vendor/bin/phpunit --coverage-clover ./.coverage/clover.xml --verbose
# TODO: The linter and code coverage must be re-enabled once PHP 8 is properly supported by those tools
#- name: Run Linter
# run: composer run-script lint-check
#- name: Upload Coverage Results To Coveralls
# env:
# COVERALLS_REPO_TOKEN: ${{ secrets.GITHUB_TOKEN }}
# run: |
# composer global require twinh/php-coveralls
# php-coveralls --coverage_clover=./.coverage/clover.xml --json_path=./coveralls-upload.json -v
- name: Upload Coverage Results To Coveralls
env:
COVERALLS_REPO_TOKEN: ${{ secrets.GITHUB_TOKEN }}
run: |
composer global require php-coveralls/php-coveralls
php-coveralls --coverage_clover=./.coverage/clover.xml --json_path=./coveralls-upload.json -v

0 comments on commit 4fdb423

Please sign in to comment.