consider consolidating health check builds

dart-lang / ecosystem

This repository is home to general Dart Ecosystem tools and packages.

BSD 3-Clause "New" or "Revised" License

47 stars 7 forks source link

consider consolidating health check builds #307

Open devoncarew opened 1 month ago

devoncarew commented 1 month ago

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:

cc @mosuem

mosuem commented 1 month ago

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

devoncarew commented 1 month ago

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.

mosuem commented 1 month ago

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?

devoncarew commented 1 month ago

Something like:

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.