[Bf-funboard] UI talk + session at bconf
ruth
ruth at ivimey.org
Fri Sep 27 01:14:39 CEST 2013
David wrote:
> Changing the mentality about code-reviews in blender, now that the
> release process is improved, will help more efficiently leverage this
> work from external developers, and also encourage them to make more
> contributions by helping them feel their work is appreciated instead
> of needless sitting on the vine.
I have always felt the value of code reviews was twofold; you get
someone who can code to attempt to pick holes, and the code author gets
to explain stuff to that reviewer, which may well prompt the "duh"
moment when they realize something won't work.
The second is the main reason I don't believe in code reviews where the
reviewer does their thing independently of the author: you lose 90% of
the benefit.
Finally, any system of code review that is stifled because people are
unhappy about the responsibility of signing off on code they don't know
well is, surely, broken too.
Regards
Ruth
More information about the Bf-funboard
mailing list