Changeset 46290 for trunk/.travis.yml

Timestamp:

09/25/2019 01:46:55 PM (6 years ago)

Author:

desrosj

Message:

Build/Test Tools: Introduce automated PHP compatibility checking.

This change introduces a new Composer script, compat that will scan the codebase for (detectable) potential PHP compatibility issues using the PHP_CodeSniffer and a custom ruleset based off of the PHPCompayibilityWP ruleset (phpcompat.xml.dist).

The command will be run as a separate job within each Travis build. While many compatibility issues and false positives have already been corrected in this commit and other Trac tickets, there are still some remaining. For that reason, the job is allowed to fail while the remainder of the potential compatibility issues are investigated and addressed. After those are resolved, the job should be set as required to pass to help prevent new compatibility issues from being introduced.

Props desrosj, jrf, all PHPCompatibilityWP and PHPCompatibility contributors.
Fixes #46152.

File:

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

trunk/.travis.yml

@@ -30 +30 @@
   - env: WP_TRAVISCI=travis:js WP_INSTALL=false
     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.3-fpm WP_TRAVISCI=test:php
     name: "PHPUnit Tests: PHP 7.3"
@@ -47 +49 @@
     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=7.4-fpm WP_TRAVISCI=test:php
   - env: LOCAL_PHP=8.0-fpm WP_TRAVISCI=test:php
@@ -121 +124 @@
       docker-compose run --rm php composer lint:errors &&
       docker-compose run --rm php composer lint tests
+    elif [[ "$WP_TRAVISCI" == "test:compat" ]]; then
+      docker-compose run --rm php composer compat
     else
       npm run grunt $WP_TRAVISCI