Skip to content
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

Update Storybook updates to v6.5.10 #49793

Merged
merged 18 commits into from
Oct 3, 2022
Merged

Update Storybook updates to v6.5.10 #49793

merged 18 commits into from
Oct 3, 2022

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented May 30, 2022

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
@storybook/addon-a11y (source) 6.4.21 -> 6.5.12 age adoption passing confidence
@storybook/addon-actions (source) 6.4.21 -> 6.5.12 age adoption passing confidence
@storybook/addon-docs (source) 6.4.21 -> 6.5.12 age adoption passing confidence
@storybook/addon-essentials (source) 6.4.21 -> 6.5.12 age adoption passing confidence
@storybook/addon-storysource (source) 6.4.21 -> 6.5.12 age adoption passing confidence
@storybook/addons (source) 6.4.21 -> 6.5.12 age adoption passing confidence
@storybook/api (source) 6.4.21 -> 6.5.12 age adoption passing confidence
@storybook/builder-webpack5 (source) 6.4.21 -> 6.5.12 age adoption passing confidence
@storybook/client-api (source) 6.4.21 -> 6.5.12 age adoption passing confidence
@storybook/components (source) 6.4.21 -> 6.5.12 age adoption passing confidence
@storybook/core-events (source) 6.4.21 -> 6.5.12 age adoption passing confidence
@storybook/manager-webpack5 (source) 6.4.21 -> 6.5.12 age adoption passing confidence
@storybook/react (source) 6.4.21 -> 6.5.12 age adoption passing confidence
@storybook/theming (source) 6.4.21 -> 6.5.12 age adoption passing confidence

Configuration

📅 Schedule: Branch creation - "before 3am on the first day of the month" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.

🔕 Ignore: Close this PR and you won't be reminded about these updates again.


  • If you want to rebase/retry this PR, click this checkbox. ⚠ Warning: custom changes will be lost.

This PR has been generated by Mend Renovate. View repository job log here.

@renovate renovate bot requested a review from a team as a code owner May 30, 2022 08:54
@renovate renovate bot requested review from dprokop and removed request for a team May 30, 2022 08:54
@renovate renovate bot added area/frontend dependencies Pull requests that update a dependency file labels May 30, 2022
@grafanabot
Copy link
Contributor

@jackw jackw requested review from jackw and removed request for dprokop May 30, 2022 09:09
@renovate renovate bot requested a review from a team as a code owner May 30, 2022 15:47
@grafanabot
Copy link
Contributor

@grafanabot
Copy link
Contributor

@jackw jackw added no-backport Skip backport of PR no-changelog Skip including change in changelog/release notes labels Aug 4, 2022
@jackw jackw added this to the 9.2.0 milestone Aug 4, 2022
@jackw jackw changed the title Update Storybook updates to v6.5.6 Update Storybook updates to v6.5.10 Aug 4, 2022
@jackw jackw force-pushed the renovate/storybook-updates branch from 4a769c3 to 446ceae Compare August 30, 2022 12:31
@grafanabot
Copy link
Contributor

@jackw
Copy link
Contributor

jackw commented Sep 6, 2022

This upgrade is currently blocked by: hipstersmoothie/storybook-dark-mode/pull/187

@jackw jackw requested review from a team, joshhunt, JoaoSilvaGrafana, mdvictor and supilee and removed request for a team September 29, 2022 12:29
@grafanabot
Copy link
Contributor

@grafanabot
Copy link
Contributor

@jackw jackw removed their request for review September 30, 2022 07:39
Copy link
Contributor

@ashharrison90 ashharrison90 left a comment

Choose a reason for hiding this comment

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

looks great to me 🙌

@@ -0,0 +1,49 @@
{
Copy link
Contributor

Choose a reason for hiding this comment

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

only thing that makes me a little queasy is this is a large amount of custom babel settings, buuuuuut i'm assuming there's no way round it? 😅

Copy link
Contributor

@jackw jackw Sep 30, 2022

Choose a reason for hiding this comment

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

Yeah, I'm not too keen on it either. I was hoping the storybook default babelrc would handle things but without this storybook cannot compile grafana/ui. Throws lots of errors related to not understanding typescript?!

If you have already enabled that plugin (or '@babel/preset-typescript'), make sure that it runs before any plugin related to additional class features:
 - @babel/plugin-proposal-class-properties
 - @babel/plugin-proposal-private-methods
 - @babel/plugin-proposal-decorators
  57 |   implements OptionsFormat
  58 | {
> 59 |   declare readonly onlyIn: (node: Node) => boolean;
     |   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  60 |   declare readonly getSyntax: (node: Node) => string;
  61 |   declare readonly renderDecoration: (
  62 |     {

Let me have a dabble and see if I can figure out a better approach. 😄

Copy link
Contributor

Choose a reason for hiding this comment

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

@ashharrison90 please see f55d927

Copy link
Contributor

Choose a reason for hiding this comment

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

@jackw anything that says turbo-build sounds good to me 😂

Copy link
Contributor

Choose a reason for hiding this comment

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

Shame it broke the build of core. I'll fix it up shortly.

Copy link
Contributor

@mckn mckn left a comment

Choose a reason for hiding this comment

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

LGTM!

@grafanabot
Copy link
Contributor

@jackw
Copy link
Contributor

jackw commented Oct 3, 2022

Leaving this here for future reference (renovate bot deletes any updates I make to PR description). I took over this PR as updating storybook to 6.5.x was more convoluted that originally expected. Actual changes in this PR:

  • Updates Storybook from 6.4.21 -> 6.5.12
  • Update Storybook configuration to remove dependencies on babel-loader, css-minimizer-webpack-plugin, postcss, postcss-loader, raw-loader, terser-webpack-plugin, ts-loader and webpack-filter-warnings-plugin
  • Use storybook-preset-scss
  • Use storybook-turbo-build to replace default storybook babel-loader with esbuild-loader
  • Introduce manager.ts for setting UI config (as per storybook docs)
  • Patch storybook-dark-mode to work with 6.5.x and yarn pnp. There's an open PR to resolve this issue which the patch is based on.
  • Enable MDXv2 for docs (what's new)
  • Fixes roboto font usage when viewing docs
  • Fixes Icon story layout where half story is hidden by storybooks ui. Bug can be seen in current canary storybook
  • Removes preview-head.html in favour of using emotion global #root styles in withTheme
  • Enable webpack5 file caching for faster subsequent start ups.
Cold cache Warm cache
Screenshot 2022-09-29 at 17 09 31 Screenshot 2022-09-29 at 17 10 15

@jackw jackw merged commit 6856784 into main Oct 3, 2022
@jackw jackw deleted the renovate/storybook-updates branch October 3, 2022 07:23
@leandro-deveikis leandro-deveikis modified the milestones: 9.3.0, 9.3.0-beta1 Nov 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/frontend dependencies Pull requests that update a dependency file enterprise-ok no-backport Skip backport of PR no-changelog Skip including change in changelog/release notes
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

7 participants