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

build(circleci): Save and link to CircleCI artifact #951

Merged
merged 3 commits into from
Feb 24, 2021

Conversation

ahobson
Copy link
Contributor

@ahobson ahobson commented Feb 22, 2021

Summary

Deploy storybook as an artifact in CircleCI and link to it from a GitHub deployment

Related Issues or PRs

How To Test

Look at the PR and follow the link to the deployment

Screenshots (optional)

@trussworks-infra-zz trussworks-infra-zz requested a deployment to storybook February 22, 2021 19:42 Abandoned
@trussworks-infra-zz trussworks-infra-zz temporarily deployed to storybook February 22, 2021 19:46 Inactive
@trussworks-infra-zz trussworks-infra-zz temporarily deployed to storybook February 22, 2021 19:54 Inactive
@trussworks-infra-zz trussworks-infra-zz temporarily deployed to storybook February 22, 2021 19:59 Inactive
* Save storybook as artifact and link GitHub deployment to it
@ahobson ahobson force-pushed the adh-storybook-artifact branch from c0ffe49 to f6783c4 Compare February 22, 2021 20:02
@trussworks-infra-zz trussworks-infra-zz temporarily deployed to storybook February 22, 2021 20:05 Inactive
@ahobson ahobson changed the title WIP artifact exporation build(circleci): Save and link to CircleCI artifact Feb 22, 2021
@trussworks-infra-zz trussworks-infra-zz temporarily deployed to storybook February 22, 2021 21:04 Inactive
@ahobson ahobson marked this pull request as ready for review February 22, 2021 21:40
@ahobson
Copy link
Contributor Author

ahobson commented Feb 22, 2021

The Happo task also builds storybook, but I'm not sure how to save the copy it builds, so we're doing 2x the work for now. If it becomes a bigger deal, we can look into fixing/optimizing things

@suzubara
Copy link
Contributor

The Happo task also builds storybook, but I'm not sure how to save the copy it builds, so we're doing 2x the work for now. If it becomes a bigger deal, we can look into fixing/optimizing things

Just linking this open issue, which is relevant and I remembered seeing before: happo/happo.io#86

@trussworks-infra-zz trussworks-infra-zz temporarily deployed to storybook February 23, 2021 18:44 Inactive
Copy link
Contributor

@brandonlenz brandonlenz left a comment

Choose a reason for hiding this comment

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

My limited CI knowledge says this looks good. I'd recommend getting 👀 from someone more well versed in this if time permits, but I won't hold it up.

@ahobson
Copy link
Contributor Author

ahobson commented Feb 24, 2021

My limited CI knowledge says this looks good. I'd recommend getting 👀 from someone more well versed in this if time permits, but I won't hold it up.

I worked with @eeeady on this, so I think we're good to go

@ahobson ahobson merged commit a536bda into main Feb 24, 2021
@ahobson ahobson deleted the adh-storybook-artifact branch February 24, 2021 21:53
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.

4 participants