[MDL-53061] Grunt watch and various directory handling improvements (backport of MDL-49817) - Moodle Tracker

XML

Word

Printable

Affected Branches:
MOODLE_29_STABLE, MOODLE_30_STABLE
Fixed Branches:
MOODLE_29_STABLE, MOODLE_30_STABLE
Pull from Repository:
git://github.com/danpoltawski/moodle.git
Difficulty:
Easy
Testing Instructions:
Hide

Run npm install to get updated grunt requirements

Run grunt watch and change many different files (amd js, yui js, bootstrapbase .less} files and ensure they are rebuilt correctly

Run other grunt commands and ensure they work as expected
Show
Run npm install to get updated grunt requirements Run grunt watch and change many different files (amd js, yui js, bootstrapbase .less} files and ensure they are rebuilt correctly Run other grunt commands and ensure they work as expected

In ~~MDL-49817~~ I added support for 'grunt watch' which allows files to be built on save.

I also simplified a lot of the grunt code and improved the 'in directory handling' which fixed ~~MDL-53059~~ and improved some other cases too.

I want to propose this is backported.

Reasons for backporting:

It's a developer tool - no impact to users, it either works or doesn't (a bit like our testing infra improvements)
To fix the other 'directory handing' bugs i'd be inclined to do the same cleanup i've done in ~~MDL-49817~~ (main motivation for creating this request)
It's useful watch functionality and keeps commands consistent between versions
Any regressions would impact the developer when building master anyway

Reasons against backporting:

It'll require an `npm install` by developers
You probably would only use `grunt watch` on one branch (master likely) then build on backport
The gruntfile has been significantly refactored

is blocked by

MDL-49817 Development - Watch for JS/LESS changes and compile once a file is saved

will help resolve

MDL-53059 Grunt fails to build YUI modules when in a yui subdirectory, builds AMD instead

Component watchers:: David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo