Skip to content

Commit

Permalink
Add custom metro cfg support for create-container
Browse files Browse the repository at this point in the history
  • Loading branch information
friederbluemle authored and belemaire committed Nov 25, 2020
1 parent 49cfa3d commit 13a2273
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 7 deletions.
16 changes: 9 additions & 7 deletions ern-local-cli/src/commands/create-container.ts
Expand Up @@ -183,15 +183,16 @@ Output directory should either not exist (it will be created) or should be empty
'You cannot create a container for a non-existing native application version.',
},
});
const compositeGenConfig = await cauldron.getCompositeGeneratorConfig(
descriptor,
);
baseComposite =
baseComposite ||
(compositeGenConfig?.baseComposite &&
PackagePath.fromString(compositeGenConfig.baseComposite));
}

const compositeGenConfig = await cauldron.getCompositeGeneratorConfig(
descriptor,
);
baseComposite =
baseComposite ||
(compositeGenConfig?.baseComposite &&
PackagePath.fromString(compositeGenConfig.baseComposite));

if (!descriptor && miniapps) {
platform = platform || (await askUserToSelectAPlatform());

Expand All @@ -205,6 +206,7 @@ Output directory should either not exist (it will be created) or should be empty
runLocalCompositeGen({
baseComposite,
jsApiImpls,
metroExtraNodeModules: compositeGenConfig?.metroExtraNodeModules,
miniApps: miniapps,
outDir: compositeDir,
}),
Expand Down
3 changes: 3 additions & 0 deletions ern-orchestrator/src/composite.ts
Expand Up @@ -15,12 +15,14 @@ export async function runLocalCompositeGen(
{
baseComposite,
jsApiImpls,
metroExtraNodeModules,
miniApps,
outDir,
resolutions,
}: {
baseComposite?: PackagePath;
jsApiImpls?: PackagePath[];
metroExtraNodeModules?: { [pkg: string]: string };
miniApps: PackagePath[];
outDir: string;
resolutions?: { [pkg: string]: string };
Expand All @@ -31,6 +33,7 @@ export async function runLocalCompositeGen(
GeneratedComposite.generate({
baseComposite,
jsApiImplDependencies: jsApiImpls,
metroExtraNodeModules,
miniApps,
outDir,
resolutions,
Expand Down

0 comments on commit 13a2273

Please sign in to comment.