infoderm / patients

:face_with_thermometer: Patients meteor app
GNU Affero General Public License v3.0
5 stars 2 forks source link

Use `workflow_run` trigger instead of `ArcticLampyrid/action-wait-for-workflow` #900

Open make-github-pseudonymous-again opened 4 months ago

make-github-pseudonymous-again commented 4 months ago

See https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#workflow_run. We could even read artifacts produced by the triggering workflow, see https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#using-data-from-the-triggering-workflow. One draw-back is that we will not be able to test this works in a PR because workflow_run-triggered workflows only run when they exist on the default branch.

make-github-pseudonymous-again commented 3 months ago

We could also consider not waiting at all and instead use the cache from previous run. Or we could wait a limited amount of time, but fallback to caches from previous run if we reach the timeout instead of failing.