Ticket #47380: 47380.2.diff

.travis.yml

@@ -89 +89 @@
 - npm --version
 - node --version
 - nvm install 10.13.0
-- npm install -g grunt-cli
 - npm install
 - npm prune
 - mysql --version
@@ -101 +100 @@
 - which phpunit
 - phpunit --version
 - curl --version
-- grunt --version
 - git --version
 - svn --version
 - locale -a
-script: grunt $WP_TRAVISCI
+script: npm run grunt $WP_TRAVISCI
 after_script:
 - |
   if [[ "$WP_TEST_REPORTER" == "true" ]]; then

appveyor.yml

@@ -11 +11 @@
 install:
   - ps: Install-Product node $env:nodejs_version
   - npm install npm -g
-  - npm install grunt-cli -g
   - npm install
 
 before_build:
@@ -19 +18 @@
   - npm --version
 
 build_script:
-  - cmd: "grunt build"
+  - cmd: "npm run build"
 
 test: off
 

package.json

@@ -113 +113 @@
                 "whatwg-fetch": "3.0.0"
         },
         "scripts": {
+                "build": "grunt build",
+                "test": "grunt test",
+                "watch": "grunt watch",
                 "grunt": "grunt"
         }
 }

src/index.php

@@ -34 +34 @@
 
 $die .= '<p>' . sprintf(
         /* translators: %s: npm install */
-        __( 'Before running any grunt tasks you need to make sure the dependencies are installed. You can install these by running %s.' ),
+        __( 'Before running any build tasks you need to make sure the dependencies are installed. You can install these by running %s.' ),
         '<code style="color: green;">npm install</code>'
 ) . '</p>';
 
 $die .= '<ul>';
 $die .= '<li>' . __( 'To build WordPress while developing, run:' ) . '<br /><br />';
-$die .= '<code style="color: green;">grunt build --dev</code></li>';
+$die .= '<code style="color: green;">npm run build --dev</code></li>';
 $die .= '<li>' . __( 'To build files automatically when changing the source files, run:' ) . '<br /><br />';
-$die .= '<code style="color: green;">grunt watch</code></li>';
+$die .= '<code style="color: green;">npm run watch</code></li>';
 $die .= '<li>' . __( 'To create a production build of WordPress, run:' ) . '<br /><br />';
-$die .= '<code style="color: green;">grunt build</code></li>';
+$die .= '<code style="color: green;">npm run build</code></li>';
 $die .= '</ul>';
 
 $die .= '<p>' . sprintf(
-        /* translators: 1: NPM URL, 2: Grunt URL, 3: Handbook URL */
-        __( 'This requires <a href="%1$s">NPM</a> and <a href="%2$s">Grunt</a>. <a href="%3$s">Read more about setting up your local development environment</a>.' ),
+        /* translators: 1: NPM URL, 2: Handbook URL */
+        __( 'This requires <a href="%1$s">NPM</a>. <a href="%2$s">Read more about setting up your local development environment</a>.' ),
         'https://www.npmjs.com/get-npm',
-        'https://gruntjs.com/getting-started',
         __( 'https://make.wordpress.org/core/handbook/tutorials/installing-wordpress-locally/' )
 ) . '</p>';
 

src/wp-includes/embed.php

@@ -446 +446 @@
         } else {
                 /*
                  * If you're looking at a src version of this file, you'll see an "include"
-                 * statement below. This is used by the `grunt build` process to directly
+                 * statement below. This is used by the `npm run build` process to directly
                  * include a minified version of wp-embed.js, instead of using the
                  * file_get_contents() method from above.
                  *
@@ -1008 +1008 @@
         } else {
                 /*
                  * If you're looking at a src version of this file, you'll see an "include"
-                 * statement below. This is used by the `grunt build` process to directly
+                 * statement below. This is used by the `npm run build` process to directly
                  * include a minified version of wp-oembed-embed.css, instead of using the
                  * readfile() method from above.
                  *
@@ -1039 +1039 @@
         } else {
                 /*
                  * If you're looking at a src version of this file, you'll see an "include"
-                 * statement below. This is used by the `grunt build` process to directly
+                 * statement below. This is used by the `npm run build` process to directly
                  * include a minified version of wp-embed-template.js, instead of using the
                  * readfile() method from above.
                  *

src/wp-includes/formatting.php

@@ -5518 +5518 @@
 
                 /*
                  * If you're looking at a src version of this file, you'll see an "include"
-                 * statement below. This is used by the `grunt build` process to directly
+                 * statement below. This is used by the `npm run build` process to directly
                  * include a minified version of wp-emoji-loader.js, instead of using the
                  * readfile() method from above.
                  *
@@ -5734 +5734 @@
  * Returns arrays of emoji data.
  *
  * These arrays are automatically built from the regex in twemoji.js - if they need to be updated,
- * you should update the regex there, then run the `grunt precommit:emoji` job.
+ * you should update the regex there, then run the `npm run grunt precommit:emoji` job.
  *
  * @since 4.9.0
  * @access private

tests/phpunit/includes/bootstrap.php

@@ -47 +47 @@
 }
 
 if ( defined( 'WP_RUN_CORE_TESTS' ) && WP_RUN_CORE_TESTS && ! is_dir( ABSPATH ) ) {
-        echo "ERROR: The /build/ directory is missing! Please run `grunt build` prior to running PHPUnit.\n";
+        echo "ERROR: The /build/ directory is missing! Please run `npm run build` prior to running PHPUnit.\n";
         exit( 1 );
 }
 

tests/phpunit/tests/oembed/template.php

@@ -309 +309 @@
          * So this test checks for ampersands in build/wp-includes/js/wp-embed.min.js.
          * In many cases, this file will not exist; in those cases, we simply skip the test.
          *
-         * So when would it be run? We have Travis CI run `grunt test` which then runs, in order,
+         * So when would it be run? We have Travis CI run `npm run test` which then runs, in order,
          * `qunit:compiled` (which runs the build) and then `phpunit`. Thus, this test will at least be
          * run during continuous integration.
          *