Skip to content

Commit

Permalink
fix(cli): create-react-app framework detection should not look for re…
Browse files Browse the repository at this point in the history
…act-dev-utils (#4585)
  • Loading branch information
elylucas committed May 18, 2021
1 parent d7f06be commit 9f7910e
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
1 change: 0 additions & 1 deletion cli/src/framework-configs.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ const FRAMEWORK_CONFIGS: FrameworkConfig[] = [
name: 'Create React App',
isMatch: config =>
hasDependency(config, 'react-scripts') &&
hasDependency(config, 'react-dev-utils') &&
!hasDependency(config, '@ionic/react'),
webDir: 'build',
},
Expand Down
3 changes: 1 addition & 2 deletions cli/test/framework-detection.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -39,9 +39,8 @@ describe('framework detection', () => {
expect(f?.webDir).toBe('dist');
});

it('Create React App', () => {
it.only('Create React App', () => {
addDep(config, 'react-scripts');
addDep(config, 'react-dev-utils');
const f = detectFramework(config);
expect(f?.name).toBe('Create React App');
expect(f?.webDir).toBe('build');
Expand Down

0 comments on commit 9f7910e

Please sign in to comment.