Remove the dependency on globally installed Grunt

[johnbillion]

There's no need for Grunt to be installed globally in order to run the build tasks because package.json already includes "grunt": "grunt" in its scripts property, which means npm run grunt [task] can be called and the local Grunt binary will be used.

[johnbillion]

@netweb Fancy giving 47380.diff​ a sanity check?

[johnbillion]

47380.2.diff​ goes a step further and introduces shortened npm run [build|test|watch] commands.

[SergeyBiryukov]

Looks good to me.

[johnbillion]

47380.3.diff​ introduces and switches to using npm run dev because I forgot that npm run foo --bar won't pass the associative argument through without the -- separator.

[netweb]

Related: #43731 - Use Webpack + NPM scripts to build all the things

This change here will be a good precursor to the npm changes proposed in #43731.

The following handbook pages will require updating:

• ​https://make.wordpress.org/core/handbook/best-practices/commit-messages/
• ​https://make.wordpress.org/core/handbook/best-practices/coding-standards/javascript/
• ​https://make.wordpress.org/core/handbook/testing/patch/
• ​https://make.wordpress.org/core/handbook/testing/automated-testing/phpunit/

[johnbillion]

I've updated all those handbook pages to use the npm run grunt ... syntax as the commands can all be run without a globally installed Grunt.

[johnbillion]

In 45445:

Build/Test Tools: Remove the dependency on a globally installed Grunt.

The existing "grunt": "grunt" script in package.json allows for the use of npm run grunt ... which uses the local grunt binary in the project's node_modules.

Fixes #47380

[johnbillion]

In 45446:

Build/Test Tools: Revert an accidental change to Node dependencies.

See #47380

[desrosj]

Mentioned in the Miscellaneous Developer Focused Changes dev note for 5.3: ​https://make.wordpress.org/core/2019/10/15/miscellaneous-developer-focused-changes-in-5-3/

@johnbillion if you had something more verbose in mind, please just let me know and we can expand on that bullet point.

[johnbillion]

Replying to desrosj:

Mentioned in the Miscellaneous Developer Focused Changes dev note for 5.3

LGTM