Web lists-archives.com

Re: CI system maintainability




Hello,

On Thursday, 28 March 2019 09:16:11 CET Ben Cooksley wrote:
> Please note that the commits in this instance were pushed without
> review, so restrictions on merge requests wouldn't make a difference
> in this case unfortunately.

Maybe it's about time to make reviews mandatory... I know it's unpopular in 
KDE, and I advocated for "don't force a tool if you can get someone to look at 
your screen or pair with you" in the past. Clearly this compromise gets 
somewhat exploited and that's especially bad in the case of a fragile and 
central component like KDE PIM.

Regards.
-- 
Kevin Ottens, http://ervin.ipsquad.net

Attachment: signature.asc
Description: This is a digitally signed message part.