-
Notifications
You must be signed in to change notification settings - Fork 69
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
⬆️ Upgrade to Storybook 7 #4406
Conversation
thesan
commented
Jun 2, 2023
•
edited
Loading
edited
- Upgrade Storybook #4402
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
5d70498
to
8f99767
Compare
8f99767
to
1edb917
Compare
1edb917
to
3a49252
Compare
3a49252
to
6eec748
Compare
ae2c84d
to
f7fa83a
Compare
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.
This needs to be added to the Changelog / release notes:
7.0 breaking changes
Dropped support for Node 15 and below
Storybook 7.0 requires Node 16 or above. If you are using an older version of Node, you will need to upgrade or keep using Storybook 6 in the meantime.
Storybook 7 migration guide
@@ -44,7 +44,7 @@ | |||
"ts-loader": "^9.2.6", | |||
"ts-node": "^10.4.0", | |||
"typescript": "4.4.3", | |||
"webpack": "^5.64.1", |
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.
Just curious, was there are reason to downgrade webpack?
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.
Actually webpack was upgraded:
It's just that there was initially duplicated versions of webpack after upgrading storybook. yarn dedupe
would have taken care of that but IMO it's just as good to just force the major version of webpack (unless we realized later that some particular minor version breaks the build).
@traumschule nice catch I should remember to mention it in the release notes when this gets released. At the same time I don't think |