-
Notifications
You must be signed in to change notification settings - Fork 42
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
ci: Deploy storybook on gh-pages #937
Conversation
bbc22bc
to
7c100e4
Compare
7c100e4
to
648bfe1
Compare
|
||
jobs: | ||
publish-storybook: | ||
if: "${{ github.event_name == 'workflow_dispatch' || contains(github.event.pull_request.labels.*.name, 'autorelease: tagged') }}" |
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.
previously (before merging repos) we were deploying storybook manually or whenever a release happened. but now it will be deployed automatically on every push.
if we want the old behavior, we can look into it after #875. changesets has a way to run arbitrary steps after a release is made. https://github.com/changesets/action#with-publishing
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.
I think it's good to deploy storybook after every merge to main. Sometimes we forget to do that manually.
That said, I think it's good to have an option to deploy manually in case we need to do that from a branch.
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 have (will we have after future updates to this script) an option to deploy and keep storybook/html pages from a branch? Eg. dev release, v2 / 3 / etc.
By default, storybook will automatically be deployed for every PR for preview. I've added back manual publish workflow, but using it will override the default storybook at |
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.
I'm new to using GitHub actions. So I left a few questions
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.
I'm new to GitHub actions and so couldn't review the PR thoroughly. But overall, everything LGTM
Before
/
or/<pr-number>
Now (in/after this PR)
portal
(from chore: Add new portal containing links to all dev servers #936) under/
or/<pr-number>
/css
or/<pr-number>/css
/react
or/<pr-number>/react
Pending issues
gh-pages
branch even after PR is closed/merged