Merge pull request #7145 from umbraco/v8/feature/0000-Remove-non-production-mode-from-gulp

remove isProd from Gulp script

src/Umbraco.Web.UI.Client/gulp/tasks/dependencies.js

@@ -259,19 +259,18 @@ function dependencies() {
     //css, fonts and image files
     
     var assetsTask = gulp.src(config.sources.globs.assets, { allowEmpty: true });
-    if (global.isProd === true) {
-        assetsTask = assetsTask.pipe(imagemin([
-            imagemin.gifsicle({interlaced: true}),
-            imagemin.jpegtran({progressive: true}),
-            imagemin.optipng({optimizationLevel: 5}),
-            imagemin.svgo({
-                plugins: [
-                    {removeViewBox: true},
-                    {cleanupIDs: false}
-                ]
-            })
-        ]));
-    }
+    assetsTask = assetsTask.pipe(imagemin([
+        imagemin.gifsicle({interlaced: true}),
+        imagemin.jpegtran({progressive: true}),
+        imagemin.optipng({optimizationLevel: 5}),
+        imagemin.svgo({
+            plugins: [
+                {removeViewBox: true},
+                {cleanupIDs: false}
+            ]
+        })
+    ]));
+    
     assetsTask = assetsTask.pipe(gulp.dest(config.root + config.targets.assets));
     
     stream.add(assetsTask);

src/Umbraco.Web.UI.Client/gulp/tasks/removeProductionMode.js

@@ -1,22 +0,0 @@
-'use strict';
-
-var gulp = require('gulp');
-var through2 = require('through2');
-
-function createEmptyStream() {
-    var pass = through2.obj();
-    process.nextTick(pass.end.bind(pass));
-    return pass;
-}
-
-/**************************
- * Copies all angular JS files into their separate umbraco.*.js file
- **************************/
-function removeProductionMode() {
-    
-    global.isProd = false;
-    
-	return createEmptyStream();
-};
-
-module.exports = { removeProductionMode: removeProductionMode };

src/Umbraco.Web.UI.Client/gulp/util/processJs.js

@@ -23,11 +23,10 @@ module.exports = function (files, out) {
     // sort files in stream by path or any custom sort comparator
     task = task.pipe(babel())
         .pipe(sort());
-
-    if (global.isProd === true) {
-        //in production, embed the templates
-        task = task.pipe(embedTemplates({ basePath: "./src/", minimize: { loose: true } }))
-    }
+    
+    //in production, embed the templates
+    task = task.pipe(embedTemplates({ basePath: "./src/", minimize: { loose: true } }))
+    
     task = task.pipe(concat(out))
         .pipe(wrap('(function(){\n%= body %\n})();'))
         .pipe(gulp.dest(config.root + config.targets.js));

src/Umbraco.Web.UI.Client/gulpfile.js

@@ -10,8 +10,6 @@
     * and then add the exports command to add the new item into the task menu.
     */
 
-global.isProd = true;
-
 const { src, dest, series, parallel, lastRun } = require('gulp');
 
 const { dependencies } = require('./gulp/tasks/dependencies');
@@ -20,7 +18,6 @@ const { less } = require('./gulp/tasks/less');
 const { testE2e, testUnit } = require('./gulp/tasks/test');
 const { views } = require('./gulp/tasks/views');
 const { watchTask } = require('./gulp/tasks/watchTask');
-const { removeProductionMode } = require('./gulp/tasks/removeProductionMode');
 
 // Load local overwrites, can be used to overwrite paths in your local setup.
 var fs = require('fs');
@@ -41,7 +38,6 @@ try {
 // ***********************************************************
 exports.build = series(parallel(dependencies, js, less, views), testUnit);
 exports.dev = series(parallel(dependencies, js, less, views), watchTask);
-exports.fastdev = series(removeProductionMode, parallel(dependencies, js, less, views), watchTask);
 exports.watch = series(watchTask);
 // 
 exports.runTests = series(js, testUnit);