datalad / git-annex

A non-official clone of git-annex established for DataLad purposes. No PRs will be merged, but could be used to test perspective git-annex patches. Official git-annex repository: https://git.kitenet.net/index.cgi/git-annex.git/
16 stars 3 forks source link

Also test on clients and send email on a triggered build #152

Closed yarikoptic closed 1 year ago

yarikoptic commented 1 year ago

I've not tried. Might need a thoroughly careful review ;)

jwodder commented 1 year ago

This change is also needed in order for the daily report e-mail to include workflow_dispatch builds:

diff --git a/.github/workflows/tools/daily-status.py b/.github/workflows/tools/daily-status.py
index c21da03d0e..a98d571013 100644
--- a/.github/workflows/tools/daily-status.py
+++ b/.github/workflows/tools/daily-status.py
@@ -166,7 +166,7 @@ def main():
         for wfname in WORKFLOWS:
             wf = wfrepo.get_workflow(wfname)
             for run in wf.get_runs():
-                if run.status != "completed" or run.event != "schedule":
+                if run.status != "completed" or run.event not in ("schedule", "workflow_dispatch"):
                     continue
                 dt = ensure_aware(run.created_at)
                 if dt <= cutoff:
yarikoptic commented 1 year ago

all good now @jwodder ?

jwodder commented 1 year ago

@yarikoptic Yes.