delete redundant comments

Author: bugthesystem

gulpfile.js

@@ -19,13 +19,16 @@ gulp.task('tpl-cache', function () {
         .pipe(gulp.dest('app'));
 });
 
+gulp.task('watch:html', function () {
+    return gulp.watch(['app/**/*.html'], ['tpl-cache']);
+});
+
 
 gulp.task('build-dev', ['tpl-cache', 'webpack:build-dev'], function () {
     gulp.watch(['app/**/*'], ['tpl-cache', 'webpack:build-dev']);
 });
 
 gulp.task('webpack:dev-server', function (callback) {
-    gulp.watch(['app/**/*'], ['tpl-cache']);
 
     new WebpackDevServer(devCompiler, {
         contentBase: 'public',
@@ -40,13 +43,11 @@ gulp.task('webpack:dev-server', function (callback) {
 
         gutil.log('[webpack-dev-server]', 'http://localhost:8080/webpack-dev-server/index.html');
 
-        // keep the server alive or continue?
         callback();
     });
 });
 
 gulp.task('webpack:build-dev', function (callback) {
-    // run webpack
     devCompiler.run(function (err, stats) {
         if (err) throw new gutil.PluginError('webpack:build-dev', err);
         gutil.log('[webpack:build-dev]', stats.toString({
@@ -61,4 +62,4 @@ gulp.task('json-server', function () {
     require('./index');
 });
 
-gulp.task('default', ['json-server', 'webpack:dev-server']);
+gulp.task('default', ['json-server', 'watch:html', 'webpack:dev-server']);

webpack.make.js

@@ -15,19 +15,8 @@ module.exports = function makeWebpackConfig(options) {
     var BUILD = !!options.BUILD;
     var TEST = !!options.TEST;
 
-    /**
-     * Config
-     * Reference: http://webpack.github.io/docs/configuration.html
-     * This is the object where all configuration gets set
-     */
     var config = {};
 
-    /**
-     * Entry
-     * Reference: http://webpack.github.io/docs/configuration.html#entry
-     * Should be an empty object if it's generating a test build
-     * Karma will set this when it's a test build
-     */
     if (TEST) {
         config.entry = {}
     } else {
@@ -36,40 +25,23 @@ module.exports = function makeWebpackConfig(options) {
         }
     }
 
-    /**
-     * Output
-     * Reference: http://webpack.github.io/docs/configuration.html#output
-     * Should be an empty object if it's generating a test build
-     * Karma will handle setting it up for you when it's a test build
-     */
     if (TEST) {
         config.output = {}
     } else {
         config.output = {
             // Absolute output directory
             path: __dirname + '/public',
 
-            // Output path from the view of the page
-            // Uses webpack-dev-server in development
             publicPath: BUILD ? '/' : 'http://localhost:8080/',
 
-            // Filename for entry points
-            // Only adds hash in build mode
             filename: BUILD ? '[name].[hash].js' : '[name].bundle.js',
             //filename: 'bundle.js',
 
-            // Filename for non-entry points
-            // Only adds hash in build mode
             chunkFilename: BUILD ? '[name].[hash].js' : '[name].bundle.js'
             //chunkFilename: BUILD ? 'bundle.js' : 'bundle.js'
         }
     }
 
-    /**
-     * Devtool
-     * Reference: http://webpack.github.io/docs/configuration.html#devtool
-     * Type of sourcemap to use per build type
-     */
     if (TEST) {
         config.devtool = 'inline-source-map';
     } else if (BUILD) {
@@ -78,46 +50,21 @@ module.exports = function makeWebpackConfig(options) {
         config.devtool = 'eval';
     }
 
-    /**
-     * Loaders
-     * Reference: http://webpack.github.io/docs/configuration.html#module-loaders
-     * List: http://webpack.github.io/docs/list-of-loaders.html
-     * This handles most of the magic responsible for converting modules
-     */
-
-        // Initialize module
     config.module = {
         preLoaders: [],
         loaders: [{
-            // JS LOADER
-            // Reference: https://github.com/babel/babel-loader
-            // Transpile .js files using babel-loader
-            // Compiles ES6 and ES7 into ES5 code
             test: /\.js$/,
             loader: 'babel',
             exclude: /node_modules/
         }, {
-            // ASSET LOADER
-            // Reference: https://github.com/webpack/file-loader
-            // Copy png, jpg, jpeg, gif, svg, woff, woff2, ttf, eot files to output
-            // Rename the file using the asset hash
-            // Pass along the updated reference to your code
-            // You can add here any file extension you want to get copied to your output
             test: /\.(png|jpg|jpeg|gif|svg|woff|woff2|ttf|eot)$/,
             loader: 'file'
         }, {
-            // HTML LOADER
-            // Reference: https://github.com/webpack/raw-loader
-            // Allow loading html through js
             test: /\.html$/,
             loader: 'raw'
         }]
     };
 
-    // ISPARTA LOADER
-    // Reference: https://github.com/ColCh/isparta-instrumenter-loader
-    // Instrument JS files with Isparta for subsequent code coverage reporting
-    // Skips node_modules and files that end with .test.js
     if (TEST) {
         config.module.preLoaders.push({
             test: /\.js$/,
@@ -131,52 +78,24 @@ module.exports = function makeWebpackConfig(options) {
         });
     }
 
-    // CSS LOADER
-    // Reference: https://github.com/webpack/css-loader
-    // Allow loading css through js
-    //
-    // Reference: https://github.com/postcss/postcss-loader
-    // Postprocess your css with PostCSS plugins
     var cssLoader = {
         test: /\.css$/,
-        // Reference: https://github.com/webpack/extract-text-webpack-plugin
-        // Extract css files in production builds
-        //
-        // Reference: https://github.com/webpack/style-loader
-        // Use style-loader in development for hot-loading
         loader: ExtractTextPlugin.extract('style', 'css?sourceMap!postcss')
     };
 
-    // Skip loading css in test mode
     if (TEST) {
-        // Reference: https://github.com/webpack/null-loader
-        // Return an empty module
         cssLoader.loader = 'null';
     }
 
-    // Add cssLoader to the loader list
     config.module.loaders.push(cssLoader);
 
-    /**
-     * PostCSS
-     * Reference: https://github.com/postcss/autoprefixer-core
-     * Add vendor prefixes to your css
-     */
     config.postcss = [
         autoprefixer({
             browsers: ['last 2 version']
         })
     ];
 
-    /**
-     * Plugins
-     * Reference: http://webpack.github.io/docs/configuration.html#plugins
-     * List: http://webpack.github.io/docs/list-of-plugins.html
-     */
     config.plugins = [
-        // Reference: https://github.com/webpack/extract-text-webpack-plugin
-        // Extract css files
-        // Disabled when in test mode or not in build mode
         new ExtractTextPlugin('[name].[hash].css', {
             disable: !BUILD || TEST
         }),
@@ -189,10 +108,7 @@ module.exports = function makeWebpackConfig(options) {
         })
     ];
 
-    // Skip rendering index.html in test mode
     if (!TEST) {
-        // Reference: https://github.com/ampedandwired/html-webpack-plugin
-        // Render index.html
         config.plugins.push(
             new HtmlWebpackPlugin({
                 template: './public/index.html',
@@ -201,19 +117,10 @@ module.exports = function makeWebpackConfig(options) {
         )
     }
 
-    // Add build specific plugins
     if (BUILD) {
         config.plugins.push(
-            // Reference: http://webpack.github.io/docs/list-of-plugins.html#noerrorsplugin
-            // Only emit files when there are no errors
             new webpack.NoErrorsPlugin(),
-
-            // Reference: http://webpack.github.io/docs/list-of-plugins.html#dedupeplugin
-            // Dedupe modules in the output
             new webpack.optimize.DedupePlugin(),
-
-            // Reference: http://webpack.github.io/docs/list-of-plugins.html#uglifyjsplugin
-            // Minify all javascript, switch loaders to minimizing mode
             new webpack.optimize.UglifyJsPlugin()
         );
     }