Changeset 50697

Timestamp:

04/10/2021 12:05:50 PM (3 years ago)

Author:

ocean90

Message:

Build/Test Tools: Remove remaining Travis CI references.

• Remove travis: tasks in Gruntfile.js.
• Remove status badge in README.md that comes from Travis CI via shields.io.
• Remove Travis CI related skipping in the WP_UnitTestCase_Base::skipOnAutomatedBranches() test method and related Docker environment variables.

Props johnbillion, SergeyBiryukov, ocean90.
See #52161.
Fixes #52666.

Location:

trunk

Files:

• Gruntfile.js (modified) (2 diffs)
• README.md (modified) (1 diff)
• docker-compose.yml (modified) (1 diff)
• phpcs.xml.dist (modified) (1 diff)
• tests/phpunit/includes/abstract-testcase.php (modified) (2 diffs)

trunk/Gruntfile.js

@@ -1640 +1640 @@
         var flags = this.flags;
         var args = changedFiles.php;
-        if ( flags.travisErrors ) {
-            // We can check the entire codebase for coding standards errors.
-            args = [ 'lint:errors' ];
-        } else if ( flags.travisWarnings ) {
-            // We can check the tests directory for errors and warnings.
-            args = [ 'lint', 'tests' ];
-        } else {
-            args.unshift( 'lint' );
-        }
+
+        args.unshift( 'lint' );
+
         grunt.util.spawn( {
             cmd: 'composer',
@@ -1661 +1655 @@
         } );
     } );
-
-    // Travis CI tasks.
-    grunt.registerTask('travis:js', 'Runs JavaScript Travis CI tasks.', [ 'jshint:corejs', 'qunit:compiled' ]);
-    grunt.registerTask('travis:phpunit', 'Runs PHPUnit Travis CI tasks.', [ 'build', 'phpunit' ]);
-    grunt.registerTask('travis:phpcs', 'Runs PHP Coding Standards Travis CI tasks.', [ 'format:php:error', 'lint:php:travisErrors:error', 'lint:php:travisWarnings:error' ]);
 
     // Patch task.

trunk/README.md

@@ -1 +1 @@
 # WordPress
-
-[![Build Status](https://img.shields.io/travis/com/WordPress/wordpress-develop/master.svg)](https://travis-ci.com/WordPress/wordpress-develop)
 
 Welcome to the WordPress development repository! Please check out the [contributor handbook](https://make.wordpress.org/core/handbook/) for information about how to open bug reports, contribute patches, test changes, write documentation, or get involved in any way you can.

trunk/docker-compose.yml

@@ -109 +109 @@
       PHP_FPM_UID: ${PHP_FPM_UID-1000}
       PHP_FPM_GID: ${PHP_FPM_GID-1000}
-      TRAVIS_BRANCH: ${TRAVIS_BRANCH-false}
-      TRAVIS_PULL_REQUEST: ${TRAVIS_PULL_REQUEST-false}
       GITHUB_REF: ${GITHUB_REF-false}
       GITHUB_EVENT_NAME: ${GITHUB_EVENT_NAME-false}

trunk/phpcs.xml.dist

@@ -97 +97 @@
     </rule>
 
-    <!-- Exclude the build folder in the current directory, as Travis puts the checkout in a build directory. -->
+    <!-- Exclude the build folder in the current directory. -->
     <exclude-pattern type="relative">^build/*</exclude-pattern>
 

trunk/tests/phpunit/includes/abstract-testcase.php

@@ -182 +182 @@
      * Allow tests to be skipped on some automated runs.
      *
-     * For test runs on Travis/GitHub Actions for something other than trunk/master,
+     * For test runs on GitHub Actions for something other than trunk/master,
      * we want to skip tests that only need to run for master.
      */
     public function skipOnAutomatedBranches() {
-        // https://docs.travis-ci.com/user/environment-variables/#Default-Environment-Variables
-        $travis_branch       = getenv( 'TRAVIS_BRANCH' );
-        $travis_pull_request = getenv( 'TRAVIS_PULL_REQUEST' );
-
         // https://docs.github.com/en/free-pro-team@latest/actions/reference/environment-variables#default-environment-variables
         $github_event_name = getenv( 'GITHUB_EVENT_NAME' );
@@ -199 +195 @@
 
             if ( in_array( $github_event_name, $skipped, true ) || 'refs/heads/master' !== $github_ref ) {
-                $this->markTestSkipped( 'For automated test runs, this test is only run on trunk/master' );
-            }
-        } elseif ( $travis_branch && 'false' !== $travis_branch ) {
-            // We're on Travis CI.
-            if ( 'master' !== $travis_branch || 'false' !== $travis_pull_request ) {
                 $this->markTestSkipped( 'For automated test runs, this test is only run on trunk/master' );
             }