Skip to content

chore: upgrade to storybook 9 #18983

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 22 commits into from
Aug 8, 2025
Merged

chore: upgrade to storybook 9 #18983

merged 22 commits into from
Aug 8, 2025

Conversation

aslilac
Copy link
Member

@aslilac aslilac commented Jul 21, 2025

Running npx storybook@latest upgrade upgraded all the packages and ran some codemods. A lot of imports changed and packages were renamed as part of this upgrade.

The storybook/react-router integration that we use requires migrating to react-router 7, which luckily is not a big change for us. Mostly they only publish new versions under the "react-router" package now, and not "react-router-dom". Imports for this were also updated accordingly.

There were also a couple stragglers that still used the "v2 story format", which had to be migrated to v3. Yet another codemod to run, but the changes map pretty much 1-to-1.

Copy link
Collaborator

@BrunoQuaresma BrunoQuaresma left a comment

Choose a reason for hiding this comment

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

If Chromatic is good, I'm good.

@github-actions github-actions bot added the stale This issue is like stale bread. label Aug 6, 2025
@aslilac aslilac requested a review from Parkreiner as a code owner August 8, 2025 17:03
@aslilac aslilac removed the stale This issue is like stale bread. label Aug 8, 2025
@aslilac aslilac merged commit 4f1db8b into main Aug 8, 2025
32 checks passed
@aslilac aslilac deleted the lilac/storybook-9 branch August 8, 2025 20:36
@github-actions github-actions bot locked and limited conversation to collaborators Aug 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants