Re:We don’t have time for code reviews
by josephlord on Wednesday October 23, 2013 @11:47AM
(#6599431)
Yes the diffs often don't show enough context and the errors may occur outside of the diff so you should really review the whole changed function even if you don't need the debugger.
They are always attached to a branch though so you can check it out to review (and run/test as appropriate).
They are always attached to a branch though so you can check it out to review (and run/test as appropriate).