<div dir="ltr"><div dir="ltr">On Sun, Apr 20, 2025 at 8:26 AM Ben Cooksley <<a href="mailto:bcooksley@kde.org">bcooksley@kde.org</a>> wrote:</div><div class="gmail_quote gmail_quote_container"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">On Sun, Apr 20, 2025 at 1:50 AM Linus Jahn <<a href="mailto:lnj@kaidan.im" target="_blank">lnj@kaidan.im</a>> wrote:</div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Sat, 19 Apr 2025 13:23:59 +0100<br>
David Edmundson <<a href="mailto:david@davidedmundson.co.uk" target="_blank">david@davidedmundson.co.uk</a>> wrote:<br>
<br>
> >Realistically merge requests shouldn't be proposed until people are<br>
> >ready to get something reviewed and merged in...  <br>
> <br>
> That's not realistic with how it is on the frontlines. The earlier we<br>
> share feedback the better, and reviews can take many many iterations.<br>
> <br>
> There's a "CI_MERGE_REQUEST_APPROVED"  variable which seems to match<br>
> what I think we're after?<br>
> Then we're running it only when the reviewer thinks it's ready.<br>
> <br>
> David<br>
<br>
And there's also CI_MERGE_REQUEST_DRAFT, so another option would also<br>
be to at least disable some jobs for draft MRs.<br></blockquote><div><br></div><div>Looks to be used very inconsistently within KWin at least, but it is something I guess.</div><div><br></div><div>I've changed the default behaviour globally now (<a href="https://invent.kde.org/sysadmin/ci-utilities/-/commit/07133f846f000ff05d9a87b49b0f47bf90614b52" target="_blank">https://invent.kde.org/sysadmin/ci-utilities/-/commit/07133f846f000ff05d9a87b49b0f47bf90614b52</a>) with jobs on draft merge requests now requiring manual runs.</div></div></div></blockquote><div><br></div><div>For the record, that had to be reverted because it broke all merge request CI.</div><div><br></div><div>Guess the draft and approved merge request variables won't work for our purposes....</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div><br></div><div>I would still like to see the KWin developers review the list of jobs they have active though in response to my initial mail and follow up to that initial mail as that has been completely ignored to date.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Linus<br></blockquote><div><br></div><div>Cheers,</div><div>Ben </div></div></div>
</blockquote></div></div>