Build task is overwriting minified JS libraries with our uglified version

[netweb]

Sanity checking diffs:

• build-31477.diff​ - /build folder diff via 31477.patch​
• build-31477-grunt-uglify-core.diff​ - diff of verbose Grunt task grunt uglify:core --verbose

[DrewAPicture]

@jorbin: Can you please review the patch and try to determine if that's all we need to fix the problem?

[jorbin]

I don't like the idea of running uglify on built JS by default. I think we are better off adding in the exceptions we need to our rules (or even better, moving everything not WP into a proper vendors folder). My above patch does this.

[ocean90]

31477.1.diff​ is 31477.diff​ plus alphabetical order and it removes sourceMappingURL from underscore.min.js since the map isn't bundled and so produces a 404 error, see #25072 for jQuery.

[iseulde]

Works for me.

[jorbin]

In 32065:

Don't reference underscore.js source map

We don't bundle the source map, so we shouldn't reference it. This prevents a 404.

Props ocean90
See #31477

[jorbin]

In 32066:

Don't override minified libraries included in core

A number of libraries that we include in core come with minified versions that we also include in core. The build process shouldn't override them.

Fixes #31477
Props ocean90, jorbin