Skip to content

Core: Disable CRA behaviors when preset detected #7696

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 10, 2019

Conversation

mrmckeb
Copy link
Member

@mrmckeb mrmckeb commented Aug 6, 2019

Disable the built-in CRA preset for @storybook/react when the new preset is detected.

Relates to storybookjs/presets#25.

@mrmckeb mrmckeb added react presets cra Prioritize create-react-app compatibility labels Aug 6, 2019
@mrmckeb mrmckeb requested a review from shilman August 6, 2019 08:31
@mrmckeb mrmckeb requested a review from thomasbertet as a code owner August 6, 2019 08:31
@mrmckeb mrmckeb self-assigned this Aug 6, 2019
@vercel
Copy link

vercel bot commented Aug 6, 2019

This pull request is automatically deployed with Now.
To access deployments, click Details below or on the icon next to each push.

@@ -27,7 +29,7 @@ export function managerWebpack(config: Configuration) {
}

export function babelDefault(config: Configuration) {
if (!isReactScriptsInstalled()) {
if (!isReactScriptsInstalled() || !builtInPresetDisabled) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be || builtInPresetDisabled?

@mrmckeb mrmckeb force-pushed the feature/disable-cra-preset branch from f212bad to 40d3cdf Compare August 6, 2019 08:37
@vercel vercel bot temporarily deployed to staging August 6, 2019 08:37 Inactive
Copy link
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Add to docs somewhere maybe?

@shilman shilman added this to the 5.2.0 milestone Aug 6, 2019
@mrmckeb mrmckeb force-pushed the feature/disable-cra-preset branch from 40d3cdf to 38c2259 Compare August 6, 2019 08:51
@vercel vercel bot temporarily deployed to staging August 6, 2019 08:51 Inactive
@mrmckeb mrmckeb force-pushed the feature/disable-cra-preset branch from 5c1b1bf to 918ab1f Compare August 6, 2019 17:46
@vercel vercel bot temporarily deployed to staging August 6, 2019 17:46 Inactive
@mrmckeb mrmckeb changed the title Add flag to disable built-in CRA preset Disable CRA behaviors when preset detected Aug 6, 2019
@mrmckeb mrmckeb force-pushed the feature/disable-cra-preset branch from 918ab1f to 6f8ef44 Compare August 6, 2019 18:19
Copy link
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is awesome. Great work!

@mrmckeb mrmckeb force-pushed the feature/disable-cra-preset branch from 6f8ef44 to 7276957 Compare August 9, 2019 17:04
@vercel vercel bot temporarily deployed to staging August 9, 2019 17:04 Inactive
@mrmckeb mrmckeb force-pushed the feature/disable-cra-preset branch from 7276957 to 1050707 Compare August 9, 2019 17:07
@vercel vercel bot temporarily deployed to staging August 9, 2019 17:07 Inactive
@mrmckeb mrmckeb force-pushed the feature/disable-cra-preset branch from 1050707 to 012d41c Compare August 9, 2019 17:09
@mrmckeb mrmckeb force-pushed the feature/disable-cra-preset branch from 012d41c to c9ca61f Compare August 9, 2019 19:06
@vercel vercel bot temporarily deployed to staging August 9, 2019 19:06 Inactive
@mrmckeb mrmckeb force-pushed the feature/disable-cra-preset branch from c9ca61f to d377340 Compare August 9, 2019 19:28
@vercel vercel bot temporarily deployed to staging August 9, 2019 19:28 Inactive
@mrmckeb mrmckeb force-pushed the feature/disable-cra-preset branch from d377340 to dedaa6d Compare August 9, 2019 19:35
@shilman shilman changed the title Disable CRA behaviors when preset detected Core: Disable CRA behaviors when preset detected Aug 10, 2019
@shilman shilman merged commit 83b5629 into next Aug 10, 2019
@shilman shilman deleted the feature/disable-cra-preset branch August 10, 2019 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cra Prioritize create-react-app compatibility feature request presets react
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants