<div dir="ltr"><div dir="ltr">On Thu, Aug 21, 2025 at 7:02 AM Nate Graham <<a href="mailto:nate@kde.org">nate@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"><br>
<br>
On 8/20/25 12:51, Ben Cooksley wrote:<br>
> On Tue, Aug 12, 2025 at 11:58 PM Nate Graham <<a href="mailto:nate@kde.org" target="_blank">nate@kde.org</a> <br>
> <mailto:<a href="mailto:nate@kde.org" target="_blank">nate@kde.org</a>>> wrote:<br>
> <br>
> <br>
> <br>
>     On 8/11/25 04:58, Ben Cooksley wrote:<br>
>      > Unfortunately the variable in question we were trying to rely on,<br>
>      > CI_MERGE_REQUEST_APPROVED is exclusive to Enterprise Edition:<br>
>      ><br>
>      > ben@sapphire:~/workspace/gitlab> grep -ir CI_MERGE_REQUEST_APPROVED *<br>
>      > ee/app/models/ee/merge_request.rb:          variables.append(key:<br>
>      > 'CI_MERGE_REQUEST_APPROVED', value: approved?.to_s) if approved?<br>
>      ><br>
>      > As such that functionality is not available to us.<br>
>      ><br>
>      > This means the options available to us are:<br>
>      > - Using a custom label (accessible via CI_MERGE_REQUEST_LABELS)<br>
> <br>
>     What does this look like?<br>
> <br>
> <br>
> It would be like any other label on a merge request.<br>
> <br>
> See Arch for instance on <a href="https://invent.kde.org/sysadmin/repo-" rel="noreferrer" target="_blank">https://invent.kde.org/sysadmin/repo-</a> <br>
> metadata/-/merge_requests/614 <<a href="https://invent.kde.org/sysadmin/repo-" rel="noreferrer" target="_blank">https://invent.kde.org/sysadmin/repo-</a> <br>
> metadata/-/merge_requests/614> - in this case the label would be <br>
> "Approved", "Ready" or something equivalent that means a bit more than <br>
> just "Run CI"<br>
<br>
Ah, that kind of label.<br>
<br>
I doubt that will work. We did a variant of this for approval before <br>
Gitlab CE backported the optional approvals feature, and it was a <br>
complete mess. Did not work at all. I think turning off the CI for draft <br>
merge requests would be a better approach if I had to choose between <br>
those options.<br></blockquote><div><br></div><div>Just having a look at KWin at least, looks like the majority of MRs aren't draft.</div><div><br></div><div>Before I make that change, which is really just removing the force-to-manual-for-Plasma, could folks please go through outstanding MRs and mark those that aren't ready to merge as draft?</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Nate<br></blockquote><div><br></div><div>Thanks,</div><div>Ben </div></div></div>