Web lists-archives.com

Re: CI system maintainability




Hello,

On Thursday, 28 March 2019 09:41:29 CET Luca Beltrame wrote:
> In data giovedì 28 marzo 2019 09:29:22 CET, Kevin Ottens ha scritto:
> > 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.
> 
> I'm not sure I agree. I can't speak for seasoned developers, but I've found
> myself in a situation (more than once) where the fix is trivial (compile
> error, missing ";", etc) and being forced to go through review would (IMO)
> unnecessarily raise friction.

I don't fully disagree with this (although I'd have stories on nefarious typos 
even for what was supposed to be a "trivial fix"). But it becomes a question 
of trade-off, IOW "how hurtful to the project mandatory reviews?" vs "how 
hurtful to the project a central component being very regularly broken?".

I'd argue we're loosing more with the current state of PIM than we'd loose 
with mandatory reviews.

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

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