Skip to content

Commit

Permalink
Merge pull request #22109 from storybookjs/fix/hide-migration-summary…
Browse files Browse the repository at this point in the history
…-on-init

CLI: Do not show a migration summary on sb init
  • Loading branch information
shilman committed May 12, 2023
1 parent fa5c55b commit 8b941ed
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 11 deletions.
25 changes: 14 additions & 11 deletions code/lib/cli/src/automigrate/index.ts
Expand Up @@ -59,6 +59,7 @@ export const automigrate = async ({
configDir: userSpecifiedConfigDir,
renderer: rendererPackage,
skipInstall,
hideMigrationSummary = false,
}: FixOptions = {}) => {
if (list) {
logAvailableMigrations();
Expand Down Expand Up @@ -100,17 +101,19 @@ export const automigrate = async ({
await remove(TEMP_LOG_FILE_PATH);
}

const packageManager = JsPackageManagerFactory.getPackageManager({ force: pkgMgr });
const installationMetadata = await packageManager.findInstallations([
'@storybook/*',
'storybook',
]);

logger.info();
logger.info(
getMigrationSummary({ fixResults, fixSummary, logFile: LOG_FILE_PATH, installationMetadata })
);
logger.info();
if (!hideMigrationSummary) {
const packageManager = JsPackageManagerFactory.getPackageManager({ force: pkgMgr });
const installationMetadata = await packageManager.findInstallations([
'@storybook/*',
'storybook',
]);

logger.info();
logger.info(
getMigrationSummary({ fixResults, fixSummary, logFile: LOG_FILE_PATH, installationMetadata })
);
logger.info();
}

cleanup();

Expand Down
1 change: 1 addition & 0 deletions code/lib/cli/src/automigrate/types.ts
Expand Up @@ -41,6 +41,7 @@ export interface FixOptions {
configDir?: string;
renderer?: string;
skipInstall?: boolean;
hideMigrationSummary?: boolean;
}

export enum FixStatus {
Expand Down
1 change: 1 addition & 0 deletions code/lib/cli/src/initiate.ts
Expand Up @@ -342,6 +342,7 @@ async function doInitiate(options: CommandOptions, pkg: PackageJson): Promise<vo
packageManager: pkgMgr,
fixes: initFixes,
configDir: installResult?.configDir,
hideMigrationSummary: true,
});
}

Expand Down

0 comments on commit 8b941ed

Please sign in to comment.