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

chore: upgrade storybook #1308

Merged
merged 4 commits into from
May 7, 2019
Merged

Conversation

cutterbl
Copy link
Collaborator

@cutterbl cutterbl commented May 7, 2019

Upgrade Storybook dependencies and configuration/addon registration.

config.module.rules.push(rules.fonts())
config.module.rules.push(rules.images())
config.module.rules.push(rules.css())
config.module.rules.push(rules.less({ browsers }))
Copy link
Owner

Choose a reason for hiding this comment

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

this will now use the default rules, which i don't think we want...

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I've pushed an update to completely override the 'rules'. They did have a rule (from the @storybook/react package) already for the js compile, as well as some bits for handling media, but this new config more closely mirrors what you were doing before.

@jquense jquense changed the title Feature/upgrade storybook chore: upgrade storybook May 7, 2019
@jquense jquense merged commit 6adeec9 into jquense:master May 7, 2019
@jquense
Copy link
Owner

jquense commented May 7, 2019

thanks!

cutterbl added a commit to cutterbl/react-big-calendar that referenced this pull request May 7, 2019
@cutterbl cutterbl deleted the feature/upgradeStorbook branch May 8, 2019 12:35
@github-actions
Copy link

🎉 This PR is included in version 0.38.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants