Skip to content

Update to @storybook/*@6#9956

Merged
Gudahtt merged 1 commit intodevelopfrom
update-storybook-v6
Nov 30, 2020
Merged

Update to @storybook/*@6#9956
Gudahtt merged 1 commit intodevelopfrom
update-storybook-v6

Conversation

@Gudahtt
Copy link
Copy Markdown
Member

@Gudahtt Gudahtt commented Nov 30, 2020

Our Storybook dependencies have been updated to v6.1.9, from v5. This was done to address a security vulnerability in a transitive dependency of these packages (highlight.js).

The primary changes required by this Storybook update were the change in import path for the withKnobs hook, the change in background config format, and the webpack configuration. Storybook seems to work correctly.

The migration was guided by the Storybook changelog and the Storybook v6 migration guide.

There is one Storybook error remaining; it fails to load the Euclid font. This is a pre-existing error though, so we can fix it in a later PR.

The yarn.lock file was deduplicated in this PR as well, as it was required to fix various install warnings that were introduced with this update.

@Gudahtt Gudahtt force-pushed the update-storybook-v6 branch 2 times, most recently from a6d71d4 to 4bf1526 Compare November 30, 2020 18:26
Our Storybook dependencies have been updated to v6.1.9, from v5. This
was done to address a security vulnerability in a transitive dependency
of these packages (`highlight.js`).

The primary changes required by this Storybook update were the change
in import path for the `withKnobs` hook, the change in background
config format, and the webpack configuration. Storybook seems to work
correctly.

The migration was guided by the Storybook changelog[1] and the
Storybook v6 migration guide[2].

There is one Storybook error remaining; it fails to load the Euclid
font. This is a pre-existing error though, so we can fix it in a later
PR.

The `yarn.lock` file was deduplicated in this PR as well, as it was
required to fix various install warnings that were introduced with this
update.

[1]: https://github.com/storybookjs/storybook/blob/next/CHANGELOG.md
[2]: https://github.com/storybookjs/storybook/blob/next/MIGRATION.md
@Gudahtt Gudahtt force-pushed the update-storybook-v6 branch from 4bf1526 to 9c4c02c Compare November 30, 2020 18:45
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [9c4c02c]
Page Load Metrics (610 ± 41 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint35124632713
domContentLoaded3987066088541
load4007086108541
domInteractive3987056088541

@Gudahtt Gudahtt marked this pull request as ready for review November 30, 2020 19:11
@Gudahtt Gudahtt requested a review from a team as a code owner November 30, 2020 19:11
@Gudahtt Gudahtt requested a review from rekmarks November 30, 2020 19:11
Copy link
Copy Markdown
Member

@rekmarks rekmarks left a comment

Choose a reason for hiding this comment

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

LGTM!

Tested locally.

@Gudahtt Gudahtt merged commit 429847a into develop Nov 30, 2020
@Gudahtt Gudahtt deleted the update-storybook-v6 branch November 30, 2020 20:10
@github-actions github-actions bot locked and limited conversation to collaborators Nov 30, 2020
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.

3 participants