Skip to content

Commit

Permalink
Merge pull request #171 from maxmind/greg/remove-unused-coverage-deps
Browse files Browse the repository at this point in the history
Remove generation of coverage info
  • Loading branch information
marselester authored Jun 10, 2024
2 parents e8be888 + f05efb9 commit 47edbb8
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 24 deletions.
20 changes: 0 additions & 20 deletions .coveralls-phpunit.xml.dist

This file was deleted.

3 changes: 1 addition & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ jobs:
php-version: ${{ matrix.php-versions }}
extensions: "mbstring, intl, ${{ matrix.php-extensions }}"
tools: "composer, phpize"
coverage: xdebug

- name: Checkout
# We use v1 due to https://github.com/actions/checkout/issues/334
Expand Down Expand Up @@ -58,7 +57,7 @@ jobs:
run: composer install --no-progress --prefer-dist --optimize-autoloader

- name: Test with phpunit without extension
run: vendor/bin/phpunit --coverage-text
run: vendor/bin/phpunit

- name: Test with phpunit using extension
run: php -d extension=ext/modules/maxminddb.so vendor/bin/phpunit
2 changes: 0 additions & 2 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,6 @@
"require-dev": {
"friendsofphp/php-cs-fixer": "3.*",
"phpunit/phpunit": ">=8.0.0,<10.0.0",
"php-coveralls/php-coveralls": "^2.1",
"phpunit/phpcov": ">=6.0.0",
"squizlabs/php_codesniffer": "3.*",
"phpstan/phpstan": "*"
},
Expand Down

0 comments on commit 47edbb8

Please sign in to comment.