-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add missing assets in order to make Storybook work #7
Conversation
"build-storybook": "build-storybook -s ./stories/assets", | ||
"deploy-storybook": "storybook-to-ghpages" | ||
}, | ||
"storybook-deployer": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need storybook-deployer
? According to the docs it's the default: https://github.com/storybooks/storybook-deployer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just added the [skip ci]
to the commit message in order to make the process faster as we don't need that steep right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
package.json
Outdated
"test": "react-scripts-ts test --env=jsdom", | ||
"storybook": "start-storybook -p 6006 -s ./stories/assets", | ||
"build-storybook": "build-storybook" | ||
"build-storybook": "build-storybook -s ./stories/assets", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need to add -o .out
to the end of this command? https://github.com/storybooks/storybook-deployer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to the docs yes, but looks like it works well now :/
Did you want to add the required Travis config to get this happening? 😄 #13 |
We can do #13 here yeah, do you know how to do it? Otherwise we can just delay it |
@majames any idea of why build is failing? |
Looks like it was a random build failure, @majames you happy with merging this? |
Approved! Failure was probs my bad dude, when I updated travis and/or coveralls config. |
https://majames.github.io/react-audio-vis 💫