ecosystem icon indicating copy to clipboard operation
ecosystem copied to clipboard

consider consolidating health check builds

Open devoncarew opened this issue 1 year ago • 4 comments

The health check workflow runs several (~6) builds per PR; we may want to consider consolidating the checks into one build, just for a simpler presentation in the PR UI. Here's the existing PR checks UI:

Screenshot 2024-10-15 at 10 29 57 AM

cc @mosuem

devoncarew avatar Oct 15 '24 17:10 devoncarew

We split them up a while ago, to make it easier to understand which one failed. I am fine with switching back though.

mosuem avatar Oct 16 '24 08:10 mosuem

Perhaps we could run them as separate steps in a build? Or have some other way to group the output to help call out which steps failed.

devoncarew avatar Oct 16 '24 17:10 devoncarew

I normally only look at the steps which failed, which is why am fine with the current solution. I don't think there is a way to group actions, that would be ideal..

What do you mean with run them as separate steps in a build?

mosuem avatar Oct 17 '24 09:10 mosuem

Something like:

Screenshot 2024-10-17 at 9 13 56 AM

I'd have to look at how the health.yml file is set up - it might surface as having individual steps in a steps: section? It would probably mean a bit of an architectural change to the file.

devoncarew avatar Oct 17 '24 16:10 devoncarew