<div dir="ltr"><div dir="ltr">On Sat, Mar 11, 2023 at 3:19 AM David Hurka <<a href="mailto:david.hurka@mailbox.org">david.hurka@mailbox.org</a>> wrote:<br></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 Thursday, March 9, 2023 9:40:47 AM CET Méven wrote:<br>
> We could use a "stale" label for MR to allow maintainers to see the<br>
> script's results.<br>
> And even a "closing-soon" label, for MR not-update in the last 12 months.<br>
<br>
Is there a rule that all open merge requests need care?<br>
I would expect that it is enough to label an open merge request as “stale”.<br>
<br>
Merge requests are usually closed because they are bad.<br>
Stale merge requests are probably good, otherwise they would have been closed <br>
intentionally.<br></blockquote><div><br></div><div>I recall in the last few months someone mentioning a MR in #kde-devel that had been approved and just never merged, and had been sitting like that for months.</div><div>All it took to get merged was a reminder by way of comment on the MR to get it merged as the original author of the change was no longer around.</div><div><br></div><div>So yes, there is definitely value in reminders and caring for those MRs.</div><div><br></div><div>I guess the difference here is between higher activity projects - whose MRs are likely to be more well looked after - and those with lower activity. Projects with lower activity tend to involve developers who look after many different projects, and will therefore benefit from reminders. Those with higher activity are much less likely to see value from it.</div><div><br></div><div>Closing of MRs though is something that should only be done after review by the developers who run the project.</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>
David<br>
<br>
<br></blockquote><div><br></div><div>Regards,</div><div>Ben </div></div></div>