Talk:Development:Reviewing

Thoughts on review model

[6:49pm] ss|work2: Everyone should be targeting the "newbie" reviewers first and letting the sr and/or smorgan look at it after that. [6:49pm] ardissone: i agree [6:50pm] ss|work2: If it's small code, let a newbie reviewer look at it, then an sr. [6:50pm] ss|work2: If it's big code, newbie reviewer than smorgan, who might find more. [6:50pm] ardissone: or complex [6:50pm] ss|work2: That will help the newbie reviewer see what they missed as well without burdening the sr queue [6:50pm] ss|work2: Right. [6:50pm] ss|work2: s/big/complex [6:50pm] ardissone: small code can still have complex interactions