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

Add missing assets in order to make Storybook work #7

Merged
merged 5 commits into from
May 29, 2017

Conversation

zzarcon
Copy link
Member

@zzarcon zzarcon commented May 24, 2017

@zzarcon zzarcon requested a review from majames May 24, 2017 14:45
"build-storybook": "build-storybook -s ./stories/assets",
"deploy-storybook": "storybook-to-ghpages"
},
"storybook-deployer": {
Copy link
Collaborator

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

Copy link
Member Author

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?

Copy link
Collaborator

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",
Copy link
Collaborator

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

Copy link
Member Author

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 :/

@majames
Copy link
Collaborator

majames commented May 24, 2017

Did you want to add the required Travis config to get this happening? 😄 #13

majames
majames previously approved these changes May 25, 2017
@zzarcon
Copy link
Member Author

zzarcon commented May 25, 2017

We can do #13 here yeah, do you know how to do it? Otherwise we can just delay it

@zzarcon
Copy link
Member Author

zzarcon commented May 28, 2017

@majames any idea of why build is failing?

@zzarcon
Copy link
Member Author

zzarcon commented May 29, 2017

Looks like it was a random build failure, @majames you happy with merging this?

@majames
Copy link
Collaborator

majames commented May 29, 2017

Approved!

Failure was probs my bad dude, when I updated travis and/or coveralls config.

@majames majames merged commit a8d324e into master May 29, 2017
@zzarcon zzarcon deleted the chore/add_assets branch May 29, 2017 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants