From 0b80f6a56fe93bde8bdc0196eb3946ab3a21b851 Mon Sep 17 00:00:00 2001 From: Jason Bedard Date: Wed, 12 Aug 2020 14:20:33 -0700 Subject: [PATCH] fix(rollup): allow config files to override default onwarn method Fixes #2084 --- packages/rollup/index.js | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/packages/rollup/index.js b/packages/rollup/index.js index 814223678b..c16e65aea0 100644 --- a/packages/rollup/index.js +++ b/packages/rollup/index.js @@ -103,12 +103,16 @@ function extractEnvironmentVariables(vars) { // input: https://rollupjs.org/guide/en/#inputoptions-object // output: https://rollupjs.org/guide/en/#outputoptions-object async function parseCLIArgs(args) { - let inputOptions = { + // Options which the CLI args or config file can override + const defaultInputOptions = { onwarn(...warnArgs) { worker.log(...warnArgs); }, }; + // Options which can override the config file + let inputOptions = {}; + let outputOptions = {}; let configFile = null; @@ -194,6 +198,9 @@ async function parseCLIArgs(args) { delete inputOptions.output; } + // Provide default inputOptions which can be overwritten + inputOptions = {...defaultInputOptions, ...inputOptions}; + // The inputs are the rule entry_point[s] inputOptions.input = inputs;