packager: Fix typing of more Resolver options

Reviewed By: davidaurelio

Differential Revision: D4351879

fbshipit-source-id: aab3eb031ae3137d43d88ad1853aa7e6f5d165c5
This commit is contained in:
Jean Lauliac 2016-12-21 03:39:50 -08:00 committed by Facebook Github Bot
parent 23261a6745
commit 4c98641537
1 changed files with 4 additions and 25 deletions

View File

@ -13,7 +13,6 @@
const DependencyGraph = require('../node-haste');
const declareOpts = require('../lib/declareOpts');
const defaults = require('../../../defaults');
const pathJoin = require('path').join;
@ -44,25 +43,6 @@ type Options = {
watch?: boolean,
};
const getDependenciesValidateOpts = declareOpts({
dev: {
type: 'boolean',
default: true,
},
platform: {
type: 'string',
required: false,
},
unbundle: {
type: 'boolean',
default: false,
},
recursive: {
type: 'boolean',
default: true,
},
});
class Resolver {
_depGraph: DependencyGraph;
@ -116,12 +96,12 @@ class Resolver {
getDependencies(
entryPath: string,
options: {},
options: {platform: string, recursive?: boolean},
transformOptions: TransformOptions,
onProgress?: ?(finishedModules: number, totalModules: number) => mixed,
getModuleId: mixed,
): Promise<ResolutionResponse> {
const {platform, recursive} = getDependenciesValidateOpts(options);
const {platform, recursive = true} = options;
return this._depGraph.getDependencies({
entryPath,
platform,
@ -138,10 +118,9 @@ class Resolver {
});
}
getModuleSystemDependencies(options: {}): Array<Module> {
const opts = getDependenciesValidateOpts(options);
getModuleSystemDependencies({dev = true}: {dev?: boolean}): Array<Module> {
const prelude = opts.dev
const prelude = dev
? pathJoin(__dirname, 'polyfills/prelude_dev.js')
: pathJoin(__dirname, 'polyfills/prelude.js');