Changes to our Git infrastructure

Boudewijn Rempt boud at valdyas.org
Tue Jan 6 16:39:18 GMT 2015


On Tue, 6 Jan 2015, Thiago Macieira wrote:

>
> Unfortunately, as long as the tool permits line-by-line commenting, you're
> going to get nitpicking. My experience is that people are linear and will
> start reading the patch, calling out what they see when they see it.
>
> They should instead look at the big picture first and that isn't easy.
>
> See http://sarah.thesharps.us/2014/09/01/the-gentle-art-of-patch-review/

Lovely article. I know some people would object that the awful 
indentation makes the patch impossible to read, but apart from that 
specious argument, it's great advice.

I wonder if we can tweak a tool to only allow line-by-line comments after 
two high-level reviews have been written...

Boudewijn




More information about the kde-core-devel mailing list