<div dir="ltr"><div dir="ltr">On Sun, Apr 20, 2025 at 12:24 AM David Edmundson <<a href="mailto:david@davidedmundson.co.uk">david@davidedmundson.co.uk</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">>Realistically merge requests shouldn't be proposed until people are 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></blockquote><div><br></div><div>The pipeline templates are globally shared among all our projects, so not sure only running on approval is the best course of action here. </div><div><br></div><div>It might work for bigger projects where you are waiting for others to review and then will have to rebase before it merges but it will break workflows where people open merge requests to validate their work passes CI before merging or where reviews are put up for a maintainer to merge if they're happy with it (due to a smaller number of core developers, often just one or two).</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>
David<br></blockquote><div><br></div><div>Cheers,</div><div>Ben </div></div></div>