Make WordPress Core


Ignore:
Timestamp:
10/03/2022 04:01:57 PM (21 months ago)
Author:
SergeyBiryukov
Message:

Build/Test Tools: Remove PHP 8.1 and 8.2 from allowed failures.

With all known unit test failures now addressed, WordPress 6.1 aims to support PHP 8.1 and 8.2 as much as possible.

While full compatibility with PHP 8.1 and 8.2 is still a work in progress, this commit aims to actively prevent new PHP issues from being introduced in WordPress core.

All remaining known issues are deprecation notices. Please note, a deprecation notice is not an error, but rather an indicator of where additional work is needed for compatibility before PHP 9 (i.e. when the notices become fatal errors). With a deprecation notice, the PHP code will continue to work and nothing is broken.

Follow-up to [49077], [49162], [50299], [51588], [51604], [53922], [54072].

Props jrf, desrosj.
See #55652, #55656, #56009, #56681.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/.github/workflows/phpunit-tests.yml

    r54343 r54369  
    195195      - name: Run PHPUnit tests
    196196        if: ${{ matrix.php >= '7.0' }}
    197         continue-on-error: ${{ matrix.php == '8.1' || matrix.php == '8.2'  }}
    198197        run: node ./tools/local-env/scripts/docker.js run php ./vendor/bin/phpunit --verbose -c ${{ env.PHPUNIT_CONFIG }}
    199198
    200199      - name: Run AJAX tests
    201200        if: ${{ ! matrix.split_slow }}
    202         continue-on-error: ${{ matrix.php == '8.2' }}
    203201        run: node ./tools/local-env/scripts/docker.js run php ./vendor/bin/phpunit --verbose -c ${{ env.PHPUNIT_CONFIG }} --group ajax
    204202
    205203      - name: Run ms-files tests as a multisite install
    206204        if: ${{ matrix.multisite && ! matrix.split_slow }}
    207         continue-on-error: ${{ matrix.php == '8.2' }}
    208205        run: node ./tools/local-env/scripts/docker.js run php ./vendor/bin/phpunit --verbose -c tests/phpunit/multisite.xml --group ms-files
    209206
    210207      - name: Run external HTTP tests
    211208        if: ${{ ! matrix.multisite && ! matrix.split_slow }}
    212         continue-on-error: ${{ matrix.php == '8.2' }}
    213209        run: node ./tools/local-env/scripts/docker.js run php ./vendor/bin/phpunit --verbose -c phpunit.xml.dist --group external-http
    214210
    215211      # __fakegroup__ is excluded to force PHPUnit to ignore the <exclude> settings in phpunit.xml.dist.
     212      # This test group is not (yet) run against PHP 8.2 as there is no stable Xdebug version available yet for PHP 8.2.
    216213      - name: Run (Xdebug) tests
    217214        if: ${{ ! matrix.split_slow && matrix.php != '8.2' }}
Note: See TracChangeset for help on using the changeset viewer.