<div dir="ltr">Will do, thanks Aaron!</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 3, 2021 at 3:38 PM Aaron Conole <<a href="mailto:aconole@redhat.com">aconole@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Salvatore,<br>
<br>
Salvatore Daniele <<a href="mailto:sdaniele@redhat.com" target="_blank">sdaniele@redhat.com</a>> writes:<br>
><br>
> The existing github monitor code only supports a single workflow job.<br>
> This worked as an initial implementation, but some projects are<br>
> configuring multiple workflows and the monitor is unable to cope with<br>
> this.<br>
><br>
> To resolve this, we pull the workflow configurations and then ensure<br>
> that all workflows are completed, and emit results once all have<br>
> finished.  Each workflow run can now emit two 'pw|...' strings.<br>
><br>
> As part of this change, we add a new field 'test_name to the 'pw|...'<br>
> string to support passing the workflow name around.  CI systems like<br>
> Travis won't report anything here so test_name will be blank for those<br>
> systems.<br>
><br>
> Signed-off-by: Salvatore Daniele <sdaniele at <a href="http://redhat.com" rel="noreferrer" target="_blank">redhat.com</a>><br>
> ---<br>
<br>
The subject on this is misspelled ("respositories" vs "repositories")<br>
<br>
The rest of this looks okay - I'll try it out.  In the meantime, please<br>
post next version as non-rfc and if I don't find any problems, I'll<br>
apply.<br>
<br>
Thanks!<br>
<br>
</blockquote></div>