From e1570798cd64762cb6dc4897656056aaf82c6c1e Mon Sep 17 00:00:00 2001 From: Jordi Boggiano Date: Tue, 23 Feb 2021 16:49:29 +0100 Subject: [PATCH 1/2] Bump CI versions --- .github/workflows/continuous-integration.yml | 39 ++++++-------------- 1 file changed, 11 insertions(+), 28 deletions(-) diff --git a/.github/workflows/continuous-integration.yml b/.github/workflows/continuous-integration.yml index f5506dd0e..de6d4edd8 100644 --- a/.github/workflows/continuous-integration.yml +++ b/.github/workflows/continuous-integration.yml @@ -12,7 +12,6 @@ env: COMPOSER_FLAGS: "--ansi --no-interaction --no-progress --prefer-dist" COMPOSER_UPDATE_FLAGS: "" COMPOSER_TESTS_ARE_RUNNING: "1" - SYMFONY_PHPUNIT_VERSION: "8.5" SYMFONY_PHPUNIT_REMOVE_RETURN_TYPEHINT: "1" jobs: @@ -34,6 +33,7 @@ jobs: - "7.2" - "7.3" - "7.4" + - "8.0" dependencies: [locked] os: [ubuntu-latest] experimental: [false] @@ -46,23 +46,23 @@ jobs: dependencies: lowest os: ubuntu-latest experimental: false - - php-version: "7.4" + - php-version: "8.0" dependencies: highest os: ubuntu-latest experimental: false - - php-version: "7.4" + - php-version: "7.4" # 8.0 fails to start the test suite for some reason on windows os: windows-latest dependencies: locked experimental: false - - php-version: "7.4" + - php-version: "8.0" os: macos-latest dependencies: locked experimental: false - - php-version: "8.0" + - php-version: "8.1" dependencies: lowest-ignore os: ubuntu-latest experimental: true - - php-version: "8.0" + - php-version: "8.1" dependencies: highest-ignore os: ubuntu-latest experimental: true @@ -71,23 +71,6 @@ jobs: - name: "Checkout" uses: "actions/checkout@v2" - - name: "Choose PHPUnit version" - if: "!startsWith(matrix.os, 'windows')" - run: | - if [ "${{ matrix.php-version }}" = "5.3" ] || [ "${{ matrix.php-version }}" = "5.4" ] || [ "${{ matrix.php-version }}" = "5.5" ]; then - echo "SYMFONY_PHPUNIT_VERSION=4.8" >> $GITHUB_ENV; - elif [ "${{ matrix.php-version }}" = "5.6" ]; then - echo "SYMFONY_PHPUNIT_VERSION=5.7" >> $GITHUB_ENV; - elif [ "${{ matrix.php-version }}" = "7.0" ]; then - echo "SYMFONY_PHPUNIT_VERSION=6.5" >> $GITHUB_ENV; - elif [ "${{ matrix.php-version }}" = "7.1" ]; then - echo "SYMFONY_PHPUNIT_VERSION=7.5" >> $GITHUB_ENV; - elif [ "${{ matrix.php-version }}" = "8.0" ]; then - echo "SYMFONY_PHPUNIT_VERSION=9.4" >> $GITHUB_ENV; - else - echo "SYMFONY_PHPUNIT_VERSION=8.5" >> $GITHUB_ENV; - fi - - name: "Install PHP" uses: "shivammathur/setup-php@v2" with: @@ -116,12 +99,12 @@ jobs: if: "matrix.dependencies == 'locked'" run: "composer install ${{ env.COMPOSER_FLAGS }}" - - name: "Require latest PHPUnitBridge for PHP 8" - if: "matrix.php-version == '8.0'" - run: composer require --no-update --dev symfony/phpunit-bridge:^5.1 + - name: "Require latest PHPUnitBridge for PHP 8.x" + if: "startsWith(matrix.php-version, '8.')" + run: composer require --no-update --dev symfony/phpunit-bridge:^5.2 - - name: "Update Symfony's PHPUnitBridge to latest available for the current PHP always as it is not really a dependency of the project" - run: "composer update ${{ env.COMPOSER_FLAGS }} symfony/phpunit-bridge" + - name: "Update dev requirements to latest available for the current PHP even on locked builds as they are not bundled dependencies" + run: "composer update ${{ env.COMPOSER_FLAGS }} symfony/phpunit-bridge phpspec/prophecy phpdocumentor/* sebastian/* doctrine/instantiator" - name: "Run install again using composer binary from source" run: "bin/composer install ${{ env.COMPOSER_FLAGS }}" From bbd87934e9cd6aeee81e162925f6248c3f0fbd84 Mon Sep 17 00:00:00 2001 From: Jordi Boggiano Date: Tue, 23 Feb 2021 20:02:33 +0100 Subject: [PATCH 2/2] Make sure we always pass a string to strtoupper --- src/Composer/Util/Platform.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Composer/Util/Platform.php b/src/Composer/Util/Platform.php index 0028981b0..aeddce5e6 100644 --- a/src/Composer/Util/Platform.php +++ b/src/Composer/Util/Platform.php @@ -98,7 +98,7 @@ class Platform // detect msysgit/mingw and assume this is a tty because detection // does not work correctly, see https://github.com/composer/composer/issues/9690 - if (in_array(strtoupper(getenv('MSYSTEM')), array('MINGW32', 'MINGW64'), true)) { + if (in_array(strtoupper(getenv('MSYSTEM') ?: ''), array('MINGW32', 'MINGW64'), true)) { return true; }