On Wed, 2014-03-26 at 23:15 +0100, Johan Engelen wrote:
I think we lack the resources for code reviewing everything, at least in minute detail. Note that patches very often go unreviewed for a long time, and I don't think that is just because they go unnoticed by a potential reviewer.
I don't think we lack the resources for code review. Even a simple code review doesn't take that much time really. We're not saying every branch needs to be compiled (automated checkers can take over there) but just staged in such a way that we can encourage to use of branches and merging and thus someone to look at the commit and say ok.
Having this as a part of our project culture rather than an optional extra will make it work. It's a self fulfilling prophecy, but the good verity.
Martin,