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

Revert "[core] Remove Storybook" #6098

Merged
merged 1 commit into from
Sep 9, 2022
Merged

Conversation

flaviendelangle
Copy link
Member

Reverts #6040

I merged it on the wrong branch...

@flaviendelangle flaviendelangle self-assigned this Sep 9, 2022
@flaviendelangle flaviendelangle added the core Infrastructure work going on behind the scenes label Sep 9, 2022
@mui-bot
Copy link

mui-bot commented Sep 9, 2022

These are the results for the performance tests:

Test case Unit Min Max Median Mean σ
Filter 100k rows ms 473.4 860.1 659.6 683.16 148.386
Sort 100k rows ms 516.4 1,104.4 804.5 835.38 191.713
Select 100k rows ms 201.9 262.7 223.6 226.94 20.099
Deselect 100k rows ms 132.9 311.5 179.7 206.54 61.091

Generated by 🚫 dangerJS against 53a4ef0

@flaviendelangle flaviendelangle merged commit 2017161 into master Sep 9, 2022
@flaviendelangle flaviendelangle deleted the revert-6040-storybook branch September 9, 2022 10:29
oliviertassinari pushed a commit to oliviertassinari/mui-x that referenced this pull request Sep 9, 2022
Revert "[core] Remove Storybook (mui#6040)"

This reverts commit e15560c.
alexfauquette pushed a commit to alexfauquette/mui-x that referenced this pull request Sep 12, 2022
Revert "[core] Remove Storybook (mui#6040)"

This reverts commit e15560c.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Infrastructure work going on behind the scenes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants