From 171c8865709281d228a934f76455f435031d80c4 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Wed, 16 Oct 2019 13:06:11 +0200 Subject: [PATCH] Silence the Autoprefixer message being printed with all build logs The following is printed with every build, which gets kind of annoying when looking at the logs: ``` Replace Autoprefixer browsers option to Browserslist config. Use browserslist key in package.json or .browserslistrc file. Using browsers option cause some error. Browserslist config can be used for Babel, Autoprefixer, postcss-normalize and other tools. If you really need to use option, rename it to overrideBrowserslist. Learn more at: https://github.com/browserslist/browserslist#readme https://twitter.com/browserslist ``` Given how we're using Autoprefixer, with a number of build-specific configs, simply changing the option name seems like the easiest solution here. (I'm also adding a couple of newlines at the `autoprefixer` call-sites, to aid readability.) --- gulpfile.js | 20 ++++++++++++++------ 1 file changed, 14 insertions(+), 6 deletions(-) diff --git a/gulpfile.js b/gulpfile.js index df71613f5879b..553467a41fe25 100644 --- a/gulpfile.js +++ b/gulpfile.js @@ -74,7 +74,7 @@ var config = JSON.parse(fs.readFileSync(CONFIG_FILE).toString()); // Default Autoprefixer config used for generic, components, minified-pre var AUTOPREFIXER_CONFIG = { - browsers: [ + overrideBrowserslist: [ 'last 2 versions', 'Chrome >= 49', // Last supported on Windows XP 'Firefox >= 52', // Last supported on Windows XP @@ -717,7 +717,9 @@ gulp.task('generic', gulp.series('buildnumber', 'default_preferences', 'locale', preprocessHTML('web/viewer.html', defines) .pipe(gulp.dest(GENERIC_DIR + 'web')), preprocessCSS('web/viewer.css', 'generic', defines, true) - .pipe(postcss([autoprefixer(AUTOPREFIXER_CONFIG)])) + .pipe(postcss([ + autoprefixer(AUTOPREFIXER_CONFIG) + ])) .pipe(gulp.dest(GENERIC_DIR + 'web')), gulp.src('web/compressed.tracemonkey-pldi-09.pdf') @@ -743,7 +745,9 @@ gulp.task('components', gulp.series('buildnumber', function () { createComponentsBundle(defines).pipe(gulp.dest(COMPONENTS_DIR)), gulp.src(COMPONENTS_IMAGES).pipe(gulp.dest(COMPONENTS_DIR + 'images')), preprocessCSS('web/pdf_viewer.css', 'components', defines, true) - .pipe(postcss([autoprefixer(AUTOPREFIXER_CONFIG)])) + .pipe(postcss([ + autoprefixer(AUTOPREFIXER_CONFIG) + ])) .pipe(gulp.dest(COMPONENTS_DIR)), ]); })); @@ -783,7 +787,9 @@ gulp.task('minified-pre', gulp.series('buildnumber', 'default_preferences', preprocessHTML('web/viewer.html', defines) .pipe(gulp.dest(MINIFIED_DIR + 'web')), preprocessCSS('web/viewer.css', 'minified', defines, true) - .pipe(postcss([autoprefixer(AUTOPREFIXER_CONFIG)])) + .pipe(postcss([ + autoprefixer(AUTOPREFIXER_CONFIG) + ])) .pipe(gulp.dest(MINIFIED_DIR + 'web')), gulp.src('web/compressed.tracemonkey-pldi-09.pdf') @@ -885,7 +891,7 @@ gulp.task('mozcentral-pre', gulp.series('buildnumber', 'default_preferences', .pipe(gulp.dest(MOZCENTRAL_CONTENT_DIR + 'web')), preprocessCSS('web/viewer.css', 'mozcentral', defines, true) .pipe(postcss([ - autoprefixer({ browsers: ['last 1 firefox versions'], }) + autoprefixer({ overrideBrowserslist: ['last 1 firefox versions'], }) ])) .pipe(gulp.dest(MOZCENTRAL_CONTENT_DIR + 'web')), @@ -935,7 +941,9 @@ gulp.task('chromium-pre', gulp.series('buildnumber', 'default_preferences', preprocessHTML('web/viewer.html', defines) .pipe(gulp.dest(CHROME_BUILD_CONTENT_DIR + 'web')), preprocessCSS('web/viewer.css', 'chrome', defines, true) - .pipe(postcss([autoprefixer({ browsers: ['chrome >= 49'], })])) + .pipe(postcss([ + autoprefixer({ overrideBrowserslist: ['chrome >= 49'], }) + ])) .pipe(gulp.dest(CHROME_BUILD_CONTENT_DIR + 'web')), gulp.src('LICENSE').pipe(gulp.dest(CHROME_BUILD_DIR)),