Skip to content

Commit

Permalink
Remove PHPUnit direct require again
Browse files Browse the repository at this point in the history
  • Loading branch information
Seldaek committed Dec 9, 2021
1 parent b94a0f8 commit 20fb797
Show file tree
Hide file tree
Showing 6 changed files with 156 additions and 1,180 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/continuous-integration.yml
Expand Up @@ -105,11 +105,11 @@ jobs:

- name: "Run tests"
if: "matrix.php-version != '7.3'"
run: "SYMFONY_PHPUNIT_VERSION=8.5 vendor/bin/simple-phpunit --verbose"
run: "vendor/bin/simple-phpunit --verbose"

- name: "Run complete test suite on 7.3"
if: "matrix.php-version == '7.3'"
run: "SYMFONY_PHPUNIT_VERSION=8.5 vendor/bin/simple-phpunit --configuration tests/complete.phpunit.xml"
run: "vendor/bin/simple-phpunit --configuration tests/complete.phpunit.xml"

validation:
name: "Composer validation"
Expand Down
13 changes: 11 additions & 2 deletions .github/workflows/phpstan.yml
Expand Up @@ -21,7 +21,8 @@ jobs:
strategy:
matrix:
php-version:
- "7.4"
- "7.2"
- "8.1"

steps:
- name: "Checkout"
Expand All @@ -46,8 +47,16 @@ jobs:
key: "php-${{ matrix.php-version }}-symfony-php-unit-version-${{ env.SYMFONY_PHPUNIT_VERSION }}-${{ hashFiles('**/composer.lock') }}"
restore-keys: "php-${{ matrix.php-version }}-symfony-php-unit-version-${{ env.SYMFONY_PHPUNIT_VERSION }}"

- name: "Install highest dependencies from composer.json using composer binary provided by system"
- name: "Install highest dependencies"
if: "matrix.php-version != '7.2'"
run: "composer config platform --unset && composer update ${{ env.COMPOSER_FLAGS }}"

- name: "Install locked dependencies"
if: "matrix.php-version == '7.2'"
run: "composer config platform --unset && composer update ${{ env.COMPOSER_FLAGS }}"

- name: "Initialize PHPUnit sources"
run: "vendor/bin/simple-phpunit --filter NO_TEST_JUST_AUTOLOAD_THANKS"

- name: "Run PHPStan"
run: "composer phpstan"
11 changes: 7 additions & 4 deletions composer.json
Expand Up @@ -21,6 +21,9 @@
"homepage": "https://seld.be"
}
],
"_comment": [
"symfony/polyfill-php73 sould be removed once https://github.com/symfony/symfony/pull/44523 is released and the symfony/console requirement bumped to match that release"
],
"require": {
"php": "^7.2.5 || ^8.0",
"composer/ca-bundle": "^1.0",
Expand All @@ -36,17 +39,17 @@
"symfony/filesystem": "^5.4 || ^6.0",
"symfony/finder": "^5.4 || ^6.0",
"symfony/process": "^5.4 || ^6.0",
"react/promise": "^2.7",
"composer/pcre": "^1.0"
"react/promise": "^2.8",
"composer/pcre": "^1.0",
"symfony/polyfill-php73": "^1.9"
},
"require-dev": {
"symfony/phpunit-bridge": "^6.0",
"phpspec/prophecy": "^1.10",
"phpstan/phpstan": "^1.0",
"phpstan/phpstan-phpunit": "^1.0",
"phpstan/phpstan-deprecation-rules": "^1",
"phpstan/phpstan-strict-rules": "^1",
"phpunit/phpunit": "^8.5"
"phpstan/phpstan-strict-rules": "^1"
},
"suggest": {
"ext-openssl": "Enabling the openssl extension allows you to access https URLs for repositories and packages",
Expand Down

0 comments on commit 20fb797

Please sign in to comment.