Changeset 48046

Timestamp:

06/15/2020 03:02:32 PM (4 years ago)

Author:

desrosj

Message:

Coding Standards: Remove the PHP Compatibility scanning job from the allowed_failures list.

All pre-existing compatibility warnings and errors as flagged by the PHPCompatibilityWP ruleset have been addressed.

Fixes #49922.

File:

• trunk/.travis.yml (modified) (2 diffs)

trunk/.travis.yml

@@ -34 +34 @@
   - env: WP_TRAVISCI=lint:php COMPOSER_INSTALL=true NPM_INSTALL=false WP_INSTALL=false
     name: PHP Linting
+  - env: WP_TRAVISCI=test:compat COMPOSER_INSTALL=true NPM_INSTALL=false WP_INSTALL=false
+    name: "PHP Compatibility Check"
   - env: WP_TRAVISCI=travis:js WP_INSTALL=false PUPPETEER_SKIP_CHROMIUM_DOWNLOAD=
     name: JS Tests
-  - env: WP_TRAVISCI=test:compat COMPOSER_INSTALL=true NPM_INSTALL=false WP_INSTALL=false
-    name: "PHP Compatibility Check"
   - env: LOCAL_PHP=7.4-fpm WP_TRAVISCI=test:php
     name: "PHPUnit Tests: PHP 7.4"
@@ -55 +55 @@
     name: "PHPUnit Tests: PHP 8.0"
   allow_failures:
-  - env: WP_TRAVISCI=test:compat COMPOSER_INSTALL=true NPM_INSTALL=false WP_INSTALL=false
   - env: LOCAL_PHP=8.0-fpm WP_TRAVISCI=test:php
   fast_finish: true