![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#47380 closed task (blessed) (fixed)
Remove the dependency on globally installed Grunt
| Reported by: | johnbillion | Owned by: | johnbillion |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.3 |
| Component: | Build/Test Tools | Version: | |
| Severity: | normal | Keywords: | has-patch commit needs-docs has-dev-note |
| Cc: | Focuses: |
Description
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.
Attachments (3)
Change History (16)
#2
@
7 years ago
47380.2.diff goes a step further and introduces shortened npm run [build|test|watch] commands.
#4
@
7 years ago
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.
#5
@
7 years ago
- Keywords commit needs-docs added
- Type enhancement → task (blessed)
#6
@
7 years ago
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.
This ticket was mentioned in Slack in #core by sergey. View the logs.
7 years ago
This ticket was mentioned in Slack in #core by sergey. View the logs.
7 years ago
#12
follow-up:
↓ 13
@
7 years ago
- Keywords has-dev-note added; needs-dev-note removed
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.
@netweb Fancy giving 47380.diff a sanity check?