Ri: Do you make code reviews?(1)
Shu: Yes. Every time we make a commit we have to ask for two reviews. And reviewers must put a comment.
Ri:You are explaining to me the formal process, which should not be confused with the quality of the actual review. Why are you doing that?
Shu: aaaa, because we must do it. It is mandatory. No commits should be made without code reviews.
Ri: So, for example, you use it for knowledge sharing?
Shu: Yes yes,exactly
Ri: Do you trust the reviews?
Shu: Of course, I can show you no commit was done without them.
Ri: That’s not what I asked you.
Shu: Yes, I trust them
Ri: What level of experience do your team members have?
Shu: hmm.ughhh…confirmed(2-3 years of experience)
Ri: And they do the review between themselves?
Shu: Yes, of course.
Ri: Hmm, how can you trust those reviews and the technical quality of that code when, actually, the persons who made them doesn’t have that much experience in writing code?
(1) Alistair Cockburn, “Shu Ha Ri”: http://alistair.cockburn.us/Shu+Ha+Ri