Azeez, can I also suggest that every 2 months you mix the people up again? Code reviews and best practices are best learnt from multiple people. That way everyone gets input from multiple leaders.
I would also like to urge the leaders to attend at least one of the other code reviews each week! Again I suggest rotating which one each person attends (no need to schedule that - each of these people should pick one and go to it). Thanks, Sanjiva. On Wed, Sep 29, 2010 at 11:58 AM, Afkham Azeez <[email protected]> wrote: > Hi team, > In order to carry out code review sessions in a more effective manner, we > will be having multiple code review sessions per week with smaller groups. > The following people will be leading the code review sessions: > > 1. Ruwan > 2. Supun > 3. Milinda > 4. Hiranya > 5. AmilaS > 6. AmilaJ > 7. Senaka > > Today we had a meeting related to this. The notes from this session are > given below: > > 1. Engineering team members have been divided into groups of 15. See [1] > 2. Groups will comprise folks from different teams, and will be a mix of > people with different experience levels. > 3. 5 reviews per week in general. We can have multiple sessions per day. > 4. We need to have descriptive code review notes, and when these issues are > addressed, a summary of what was done should be provided. > 5. We plan to evaluate the code review tool Crucible - > http://crucible.wso2.com during the first few sessions > 6. Code reviews need to be prescheduled and the relevant group of people > should be informed of the sections in advance, so that they can provide > feedback before the session. > > > Reference > 1. > https://spreadsheets0.google.com/a/wso2.com/ccc?key=tE8m7UvpY-DkW9xM596-e3Q&hl=en#gid=0 > > > > <https://spreadsheets0.google.com/a/wso2.com/ccc?key=tE8m7UvpY-DkW9xM596-e3Q&hl=en#gid=0>Thoughts > & suggestions are welcome. > > Thanks > -- > *Afkham Azeez* > Senior Software Architect & Senior Manager; WSO2, Inc.; http://wso2.com, > * > * > *Member; Apache Software Foundation; > **http://www.apache.org/*<http://www.apache.org/> > * > email: **[email protected]* <[email protected]>* cell: +94 77 3320919 > blog: **http://blog.afkham.org* <http://blog.afkham.org>* > twitter: **http://twitter.com/afkham_azeez*<http://twitter.com/afkham_azeez> > * > linked-in: **http://lk.linkedin.com/in/afkhamazeez* > * > * > *Lean . Enterprise . Middleware* > > > > > _______________________________________________ > Carbon-dev mailing list > [email protected] > https://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev > > -- Sanjiva Weerawarana, Ph.D. Founder, Chairman & CEO; WSO2, Inc.; http://wso2.com/ email: [email protected]; phone: +1 408 754 7388 x51726; cell: +94 77 787 6880 | +1 650 265 8311 blog: http://sanjiva.weerawarana.org/ Lean . Enterprise . Middleware
_______________________________________________ Carbon-dev mailing list [email protected] https://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev
