<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Kevin Ottens <<a href="mailto:ervin@kde.org">ervin@kde.org</a>> schrieb am Do., 28. März 2019, 09:29:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<br>
On Thursday, 28 March 2019 09:16:11 CET Ben Cooksley wrote:<br>
> Please note that the commits in this instance were pushed without<br>
> review, so restrictions on merge requests wouldn't make a difference<br>
> in this case unfortunately.<br>
<br>
Maybe it's about time to make reviews mandatory... </blockquote></div></div><div dir="auto"><br></div><div dir="auto">We could make it mandatory with a possible backdoor. Like if you have a line in your commit log saying "I know what I am doing." then the commit could be done without review. Of course this sounds like everyone could use this backdoor always, but I doubt this would be the case.</div><div dir="auto"><br></div><div dir="auto">Also, unreviewed commits could have a "[not reviewed]" suffix on <a href="mailto:kde-commits@kde.org">kde-commits@kde.org</a> like the License additions so that it's easier to spot unreviewed commits. We could even CC the author or respective mailing list recommending that reviews should be done for future commits.</div><div dir="auto"><br></div><div dir="auto">Just some thoughts :)</div><div dir="auto"><br></div><div dir="auto">Greetings</div><div dir="auto">Dominik</div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I know it's unpopular in <br>
KDE, and I advocated for "don't force a tool if you can get someone to look at <br>
your screen or pair with you" in the past. Clearly this compromise gets <br>
somewhat exploited and that's especially bad in the case of a fragile and <br>
central component like KDE PIM.<br>
<br>
Regards.<br>
-- <br>
Kevin Ottens, <a href="http://ervin.ipsquad.net" rel="noreferrer noreferrer" target="_blank">http://ervin.ipsquad.net</a><br>
</blockquote></div></div></div>