diff --git a/packages/cli/src/tools/config/index.js b/packages/cli/src/tools/config/index.js index 9c402b84e..704f03bee 100644 --- a/packages/cli/src/tools/config/index.js +++ b/packages/cli/src/tools/config/index.js @@ -12,7 +12,6 @@ import makeHook from './makeHook'; import { readConfigFromDisk, readDependencyConfigFromDisk, - readLegacyDependencyConfigFromDisk, } from './readConfigFromDisk'; import {type ConfigT} from 'types'; @@ -39,9 +38,7 @@ function loadConfig(projectRoot: string = process.cwd()): ConfigT { let config; try { root = resolveNodeModuleDir(projectRoot, dependencyName); - config = - readLegacyDependencyConfigFromDisk(root) || - readDependencyConfigFromDisk(root); + config = readDependencyConfigFromDisk(root); } catch (error) { logger.warn( inlineString(` diff --git a/packages/cli/src/tools/config/readConfigFromDisk.js b/packages/cli/src/tools/config/readConfigFromDisk.js index dc2688fa9..babfb3887 100644 --- a/packages/cli/src/tools/config/readConfigFromDisk.js +++ b/packages/cli/src/tools/config/readConfigFromDisk.js @@ -55,7 +55,9 @@ export function readDependencyConfigFromDisk( searchPlaces, }); - const {config} = explorer.searchSync(rootFolder) || {config: undefined}; + const {config} = explorer.searchSync(rootFolder) || { + config: readLegacyDependencyConfigFromDisk(rootFolder), + }; const result = Joi.validate(config, schema.dependencyConfig); @@ -90,7 +92,7 @@ const loadProjectCommands = ( /** * Reads a legacy configuration from a `package.json` "rnpm" key. */ -export function readLegacyDependencyConfigFromDisk( +function readLegacyDependencyConfigFromDisk( rootFolder: string, ): ?UserDependencyConfigT { const {rnpm: config, name} = require(path.join(rootFolder, 'package.json')); @@ -113,21 +115,15 @@ export function readLegacyDependencyConfigFromDisk( commands: loadProjectCommands(rootFolder, config.plugin), platforms: config.platform ? require(path.join(rootFolder, config.platform)) - : undefined, + : {}, }; // @todo: paste a link to documentation that explains the migration steps logger.warn( `Package ${chalk.bold( path.basename(name), - )} is using deprecated "rnpm" config that will stop working from next release. Consider upgrading to the new config format.`, + )} is using deprecated "rnpm" config that will stop working from next release. Please notify its maintainers about it.`, ); - const result = Joi.validate(transformedConfig, schema.dependencyConfig); - - if (result.error) { - throw new JoiError(result.error); - } - - return result.value; + return transformedConfig; }