From 930c98172c45a874003d3ed39910fc86ec8bb40e Mon Sep 17 00:00:00 2001 From: Ville Immonen Date: Mon, 6 May 2019 17:36:44 +0300 Subject: [PATCH] fix: reporter config option The `loadConfig` function in `metro-config` reads the `reporter` option from the second argument (`defaultConfigOverrides`) instead of the first argument (`argv`), so we need to pass it in that object to make the `customLogReporterPath` CLI option work. --- packages/cli/src/tools/loadMetroConfig.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/packages/cli/src/tools/loadMetroConfig.js b/packages/cli/src/tools/loadMetroConfig.js index da713adfc..882dccfd5 100644 --- a/packages/cli/src/tools/loadMetroConfig.js +++ b/packages/cli/src/tools/loadMetroConfig.js @@ -82,6 +82,8 @@ export type ConfigOptionsT = {| */ export default function load(ctx: ConfigT, options?: ConfigOptionsT) { const defaultConfig = getDefaultConfig(ctx); - - return loadConfig({cwd: ctx.root, ...options}, defaultConfig); + return loadConfig( + {cwd: ctx.root, ...options}, + {...defaultConfig, reporter: options && options.reporter}, + ); }