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

🪟 🔧 Publish webapp storybook to chromatic in CI #16169

Merged
merged 15 commits into from
Sep 8, 2022

Conversation

edmundito
Copy link
Contributor

@edmundito edmundito commented Aug 31, 2022

What

Closes #14846

Adds step to publish storybook to chromatic in CI

image

How

Uses the @chromaui/action Github action to publish the storybook during the "Frontend: Build" CI workflow step

Tests

  • CI publishes storybook!

@edmundito edmundito temporarily deployed to more-secrets August 31, 2022 14:31 Inactive
@edmundito edmundito force-pushed the edmundito/storybook-to-chromatic-workflow branch from 43ddc2b to be126fc Compare August 31, 2022 17:56
@edmundito edmundito temporarily deployed to more-secrets August 31, 2022 17:59 Inactive
@edmundito edmundito temporarily deployed to more-secrets August 31, 2022 18:22 Inactive
@edmundito edmundito temporarily deployed to more-secrets August 31, 2022 20:07 Inactive
@edmundito edmundito temporarily deployed to more-secrets August 31, 2022 20:50 Inactive
@edmundito edmundito temporarily deployed to more-secrets September 1, 2022 12:28 Inactive
@edmundito edmundito force-pushed the edmundito/storybook-to-chromatic-workflow branch 2 times, most recently from ce486b2 to 1f8999b Compare September 1, 2022 14:23
@edmundito edmundito temporarily deployed to more-secrets September 1, 2022 14:26 Inactive
@edmundito edmundito force-pushed the edmundito/storybook-to-chromatic-workflow branch from 1f8999b to c5378fb Compare September 1, 2022 16:11
@edmundito edmundito temporarily deployed to more-secrets September 1, 2022 16:13 Inactive
@edmundito edmundito temporarily deployed to more-secrets September 1, 2022 18:57 Inactive
@edmundito edmundito temporarily deployed to more-secrets September 1, 2022 19:12 Inactive
@edmundito edmundito temporarily deployed to more-secrets September 2, 2022 13:13 Inactive
@edmundito edmundito temporarily deployed to more-secrets September 2, 2022 13:38 Inactive
@edmundito edmundito marked this pull request as ready for review September 2, 2022 13:59
@edmundito edmundito added the area/frontend Related to the Airbyte webapp label Sep 2, 2022
@edmundito edmundito requested review from timroes, davinchia and a team September 2, 2022 13:59
runs-on: ${{ needs.start-frontend-runner.outputs.label }}
steps:
- name: Checkout Airbyte
uses: actions/checkout@v2
with:
Copy link
Contributor

Choose a reason for hiding this comment

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

Curious, why add this?

Copy link
Contributor Author

@edmundito edmundito Sep 7, 2022

Choose a reason for hiding this comment

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

It's for Chromatic to be able to track the history:

Chromatic supports the latest versions of the actions/checkout (i.e., versions 2 and 3). Both of them come with a caveat. They will only retrieve a single commit without any additional history. Chromatic needs the full Git history to keep track of changes in your repository.

https://www.chromatic.com/docs/github-actions#support-for-codeactionscheckoutv2code-and-above

@edmundito edmundito merged commit 81e8a51 into master Sep 8, 2022
@edmundito edmundito deleted the edmundito/storybook-to-chromatic-workflow branch September 8, 2022 18:46
robbinhan pushed a commit to robbinhan/airbyte that referenced this pull request Sep 29, 2022
* Add Publish to storybook step in webapp gradle workflow

* Set fetch-dept to 0 for airbyte webapp build

* Fix typo in chromatic config

* Fix typo when you've been writing too much css

* Fix storybook build path in gradle workflow

* Use full path in chromatic action

* Try another path configuration for chromatic workflow action

* Use label ot token in storybook deploy

* Back to github_token

* Move Storybook deploy to separate job

* Fix run command in publish storybook workflow

* Rollback publish storybook as separate step from build-frontend, add correct github token to step

* Let the chromatic step  build the storybook

* Restore storybookBuildDir on publish to chromatic step

* Add exitOnceUploaded to chromatic action
jhammarstedt pushed a commit to jhammarstedt/airbyte that referenced this pull request Oct 31, 2022
* Add Publish to storybook step in webapp gradle workflow

* Set fetch-dept to 0 for airbyte webapp build

* Fix typo in chromatic config

* Fix typo when you've been writing too much css

* Fix storybook build path in gradle workflow

* Use full path in chromatic action

* Try another path configuration for chromatic workflow action

* Use label ot token in storybook deploy

* Back to github_token

* Move Storybook deploy to separate job

* Fix run command in publish storybook workflow

* Rollback publish storybook as separate step from build-frontend, add correct github token to step

* Let the chromatic step  build the storybook

* Restore storybookBuildDir on publish to chromatic step

* Add exitOnceUploaded to chromatic action
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/frontend Related to the Airbyte webapp
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Publish storybook to chromatic
2 participants