Code-reviews as relationship builders
I really enjoyed the emphasis of Ben’s Code-reviews as relationship builders post over the weekend. The focus on healthy learning relationships both for new and old is great – something I’ve tried to focus on myself over the past few years working in distributed teams.
On a related note, I’ve wondered of late whether I could use “Agreed” rather than “Approved” when reviewing a branch. Agreed (IMO) implies one type of relationship – peer discussion resulting in agreement – rather than an approval from an authority (perhaps without interaction). It is a small and maybe insignificant distinction…
I don’t think the distinction is all that insignificant. Certainly something to consider.
We’ve run into the same thing with code review for the “Rejected” status. Which we almost never use, because it sounds quite harsh. Things tended to end up in huge “Work-In-Progress” lists, because while we weren’t happy with the current implementation, we didn’t want to “Reject” the idea.
John
March 25, 2011 at 11:19 am
True – I tend to use either “Needs information” or if it’s obviously broken “Needs fixing”. Normally (in my experience), it’s the person who submitted the MP who ‘rejects’ it if need be… (in which case, they’re really withdrawing it from discussion/consideration).
Michael
March 25, 2011 at 12:58 pm