CivicTechTO / polis-storybook

https://civictechto.github.io/polis-storybook/
GNU Affero General Public License v3.0
1 stars 0 forks source link

Don't deploy storybook when checks for success fail #24

Open patcon opened 2 weeks ago

patcon commented 2 weeks ago

Re: https://github.com/bitovi/github-actions-storybook-to-github-pages

We need to run deploy after the checks for success. Just realized in the CI run that it was deploying the the action step, and so a failure didn't matter. We want deploy in a separate step, so it doesn't happen https://github.com/CivicTechTO/polis-storybook/actions/runs/11469336432/job/31916377978

To Do

patcon commented 2 weeks ago

Also, we should have a way to check for failures in the main build. This should be rare if we use PRs, but ca n happen if we push direct to main. Maybe this is a separate ticket tho

thomassth commented 2 weeks ago

Rule added

patcon commented 1 week ago

Thanks for chipping in with issue queue gardening, but this isn't yet started. Added specific todos to make that clear 🙏