Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(652)

Unified Diff: web/apps/gulp-common.js

Issue 2969643003: [web] Update packages, use TS 2.4 (Closed)
Patch Set: [web] Update packages, use TS 2.4 Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | web/inc/logdog-stream-view/model.ts » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: web/apps/gulp-common.js
diff --git a/web/apps/gulp-common.js b/web/apps/gulp-common.js
index 39944b265a01c38728533989756a8d8f8b2de48e..be7fc0d95b7ca09408a412ef8dfb32322721c9c7 100644
--- a/web/apps/gulp-common.js
+++ b/web/apps/gulp-common.js
@@ -22,12 +22,14 @@ exports.plugins = require('gulp-load-plugins')({
// Include Gulp & tools we'll use
var $ = exports.plugins;
var browserSync = require('browser-sync');
+var cleanCSS = require('gulp-clean-css');
var debug = require('gulp-debug');
var del = require('del');
var format = require('gulp-clang-format');
var fs = require('fs');
var glob = require('glob-all');
var gulpIf = require('gulp-if');
+var htmlmin = require('gulp-htmlmin');
var historyApiFallback = require('connect-history-api-fallback');
var hyd = require('hydrolysis');
var merge = require('merge-stream');
@@ -141,7 +143,7 @@ exports.setup = function(gulp, config) {
}))
.pipe($.autoprefixer(AUTOPREFIXER_BROWSERS))
.pipe(gulp.dest('.tmp/' + stylesPath))
- .pipe($.minifyCss())
+ .pipe(cleanCSS())
.pipe(gulp.dest(layout.dist(stylesPath)))
.pipe($.size({title: stylesPath}));
};
@@ -169,14 +171,14 @@ exports.setup = function(gulp, config) {
})))
// Concatenate and minify styles
// In case you are still using useref build blocks
- .pipe($.if('*.css', $.minifyCss()))
+ .pipe($.if('*.css', cleanCSS()))
.pipe(assets.restore())
.pipe($.useref())
// Minify any HTML
- .pipe($.if('*.html', $.minifyHtml({
- quotes: true,
- empty: true,
- spare: true
+ .pipe($.if('*.html', htmlmin({
+ remoteAttributeQuotes: false,
+ remoteEmptyAttributes: false,
+ remoteRedundantAttributes: false,
})))
// Output files
.pipe(gulp.dest(dest))
« no previous file with comments | « no previous file | web/inc/logdog-stream-view/model.ts » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698