Full reviewer. Mentorship is the process by which we would entrust people with 
reviewing code and having internalized the conventions enough to make the 
right judgment call about whether a review is needed or not.

-- 
Francis J. Lacoste
[email protected]

On October 20, 2010, Steve Kowalik wrote:
> On 21/10/10 01:26, Francis J. Lacoste wrote:
> > I agree that the 3 months period is probably a bad criteria. Let's change
> > the criteria to being a reviewer. Which usually happens after being on
> > the job for 3 months.
> 
> Ah, but a full reviewer, or one that is being mentored? I would argue
> for the former, and not the latter.
> 
> Cheers,

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
Mailing list: https://launchpad.net/~launchpad-dev
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~launchpad-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to