-
Notifications
You must be signed in to change notification settings - Fork 492
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
UI: Upgrade to Storybook 8.3.0-beta.0
, add Vitest to UI package
#4802
base: master
Are you sure you want to change the base?
Conversation
@JReinhold is attempting to deploy a commit to the GitButler Team on Vercel. A member of the Team first needs to authorize it. |
@@ -8,7 +8,8 @@ const preview: Preview = { | |||
backgrounds: { disable: true }, | |||
controls: { | |||
matchers: { | |||
color: /(background|color)$/i, | |||
// matches "background" and "color" but not "solidBackground" as that is a boolean arg | |||
color: /(?<!solid)(background|color)$/i, |
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 silents a console warning that the controls addon was logging in a story that uses solidBackground
. The console warning was more noticeable in the Vitest tests because it would show up in the output.
"storybook-dark-mode": "^4.0.2", | ||
"svelte": "catalog:svelte", | ||
"svelte-check": "catalog:svelte", | ||
"vite": "catalog:" | ||
"util": "^0.12.5", |
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.
util
here is a temporary workaround for storybookjs/storybook#29008. Can be removed again when that fix has been released.
☕️ Reasoning
Storybook maintainer here. 👋
As part of our upcoming 8.3.0 release of Storybook, we're QAing it using various open source projects like this one. I figured that since I did this upgrade I might as well open a PR with it so you can benefit from it, or not.
Feel free to immediately close this PR if it's not interesting to you.
🧢 Changes
8.3.0-beta.0
Read more about the experiment here: https://web-git-8-3-pre-release-storybookjs.vercel.app/docs/writing-tests/test-runner-with-vitest
It worked flawlessly in your setup:
I added appropriate scripts which turbo picks up, but I did not add it to CI.
📌 Todos