Skip to content

Commit

Permalink
Merge pull request #25717 from storybookjs/valentin/add-prompt-remove…
Browse files Browse the repository at this point in the history
…-react

Automigration: Add removeReactDependency fix to allFixes array
  • Loading branch information
valentinpalkovic authored Jan 23, 2024
2 parents 13b30d6 + ca9d02c commit bb7434c
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 0 deletions.
2 changes: 2 additions & 0 deletions code/lib/cli/src/automigrate/fixes/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ import { incompatibleAddons } from './incompatible-addons';
import { angularBuildersMultiproject } from './angular-builders-multiproject';
import { wrapRequire } from './wrap-require';
import { reactDocgen } from './react-docgen';
import { removeReactDependency } from './prompt-remove-react';
import { storyshotsMigration } from './storyshots-migration';

export * from '../types';
Expand All @@ -42,6 +43,7 @@ export const allFixes: Fix[] = [
wrapRequire,
reactDocgen,
storyshotsMigration,
removeReactDependency,
];

export const initFixes: Fix[] = [eslintPlugin];
1 change: 1 addition & 0 deletions code/lib/cli/src/automigrate/fixes/prompt-remove-react.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ import type { Fix } from '../types';

export const removeReactDependency: Fix<{}> = {
id: 'remove-react-dependency',
promptOnly: true,

async check({ packageManager, mainConfig, storybookVersion }) {
// when the user is using the react renderer, we should not prompt them to remove react
Expand Down

0 comments on commit bb7434c

Please sign in to comment.