Skip to content

[storybook] Remove need for yarn-link#116

Merged
oliviertassinari merged 3 commits into
mui:masterfrom
oliviertassinari:no-more-yarn-link
Aug 2, 2020
Merged

[storybook] Remove need for yarn-link#116
oliviertassinari merged 3 commits into
mui:masterfrom
oliviertassinari:no-more-yarn-link

Conversation

@oliviertassinari

@oliviertassinari oliviertassinari commented Jul 27, 2020

Copy link
Copy Markdown
Member

Remove the need to yarn link or build, the storybook just work™. (I have almost never managed to make it work locally before that): 9ad13fe.

Fix #143

Comment thread packages/storybook/babel.config.js Outdated
@oliviertassinari
oliviertassinari marked this pull request as ready for review August 2, 2020 17:15
@oliviertassinari oliviertassinari added the docs Improvements or additions to the documentation. label Aug 2, 2020
@oliviertassinari
oliviertassinari force-pushed the no-more-yarn-link branch 3 times, most recently from d784f73 to eeeb793 Compare August 2, 2020 17:51
@oliviertassinari
oliviertassinari merged commit 39fcc40 into mui:master Aug 2, 2020
@oliviertassinari
oliviertassinari deleted the no-more-yarn-link branch August 2, 2020 18:02
This was referenced Aug 2, 2020
@oliviertassinari

oliviertassinari commented Aug 3, 2020

Copy link
Copy Markdown
Member Author

Note that I didn't use Babel to solve the problem, webpack aliases did most of the job.
It's soo much better now, from a workflow perspective :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to the documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants