-
Notifications
You must be signed in to change notification settings - Fork 8
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 Storybook to v8 #787
upgrade Storybook to v8 #787
Conversation
…e spacing.stories.mdx to spacing.mdx as it is not supported by storybook (it refers to an old way to build stories even if file does not use it this way)
Hello jeanmarcmilletscality,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
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.
LGTM !
/approve |
I have successfully merged the changeset of this pull request
Please check the status of the associated issue ARTESCA-13637. Goodbye jeanmarcmilletscality. The following options are set: approve |
Upgrade Storybook to v8:
Migration to Storybook V8 and update 'tests' workflow
Storybook Migration:
.storybook/main.ts
. [1] [2]ScrollbarWrapper
import and addedtags
parameter in.storybook/preview.js
. [1] [2]GitHub Actions Workflow Updates:
.github/workflows/tests.yaml
to use single quotes for consistency.actions/checkout
andactions/setup-node
versions tov4
and changed the Node.js version to20
.Code Improvements:
src/lib/components/tabsv2/Tabsv2.component.tsx
andsrc/lib/organisms/attachments/AttachmentTable.tsx
. [1] [2]