MDL-52250 npm: remove uglify-js dependency/bump grunt-contrib-uglify
authorDan Poltawski <dan@moodle.com>
Wed, 2 Dec 2015 14:18:43 +0000 (14:18 +0000)
committerDan Poltawski <dan@moodle.com>
Fri, 4 Dec 2015 15:07:47 +0000 (15:07 +0000)
commit302a3c5f7ded0de7f26f3fdaf90547bb78903e73
tree32acf0bcabdf2dab67aa5d702f6c7c106f18d474
parente90eb13a6b762b39a93ead98ffdc336c0b7b12f7
MDL-52250 npm: remove uglify-js dependency/bump grunt-contrib-uglify

A) Remove direct dependency on uglify-js:

This dependency was added in MDL-50277, but it isn't quite correct, the
way npm installs dependencies means that grunt-contrib-uglify *could* use
'our' version if its satisifed. But does not *always* and especially does
not if the grunt-contrib-uglify requirement is higher than ours.

Long story short, in many cases this means that this is used:
 node_modules/grunt-contrib-uglify/node_modules/uglify-js
But this is not:
  node_modules/uglify-js

In MDL-50277 we thought we were making the version sticky, but in
reality we were just fixing previous problems by updating
grunt-contrib-uglify to a version which fixed a bug:

 https://github.com/gruntjs/grunt-contrib-uglify/issues/313

It seems the better solution will come with using npm shrinkwrap to lock
down depdencies.

B) Bump grunt-contrib-uglify to 0.11.0

This moves us to uglify-js ~2.6.0 - which doens't change any output
files.
package.json