-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Storybook: Update to the version 6.4 #37367
Conversation
Size Change: 0 B Total Size: 1.12 MB ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No regressions noticeable for me either 👍
Probably doesn't solve the npm 8 problem you're describing, but it does seem like they have a cleaner workaround for the alias hack now:
- managerWebpack: updateEmotionAliases,
+ features: { emotionAlias: false },
Line 50 in 1786db8
managerWebpack: updateEmotionAliases, |
@mirka, would you mind opening a follow-up PR with the fix you mentioned? It looks like they want to include Emotion 11 support in Storybook 7. |
Followed up with some config migration in #37544 |
Description
Storybook 6.4 is out: https://storybook.js.org/blog/storybook-6-4/.
I hoped that they added support for Emotion v11, but it's still not resolved storybookjs/storybook#13300. I'm trying to make Gutenberg work with npm 8 and seems to have issues with 2 different versions of Emotion used in the lock file.
How has this been tested?
I executed:
npm run storybook:build
Everything looked like before but I didn't do very in-depth testing.
Types of changes
New feature (non-breaking change which adds functionality).
Checklist:
*.native.js
files for terms that need renaming or removal).