[gmx-developers] 4.6.1 is out

Erik Marklund erikm at xray.bmc.uu.se
Wed Mar 6 16:25:52 CET 2013

On Mar 6, 2013, at 2:40 PM, Shirts, Michael (mrs5pt) wrote:

>> Code reviewing can be sped up by adding reviewers. The person that
>> submits a gerrit patch usually has a decent idea of who might know  
>> the
>> code sufficiently well to review it. We should however share the load
>> and not dump everything on e.g. Berk. I usually add two to three  
>> people
>> to patches, which means they get mail at least once.
> This is something I struggle with -- I don't always know who to  
> add.  Berk
> knows integrators and free energy, Sander knows free energies . . .  
> And I
> don't know who else.
> So if we had some sort of list of who could review on what subject,  
> that
> might be good.

We could have special mailing lists or similar for certain groups of  
reviewers to simplify this. E.g. analysis_tools_review at gromacs.org.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://maillist.sys.kth.se/pipermail/gromacs.org_gmx-developers/attachments/20130306/feccbcda/attachment.html>

More information about the gromacs.org_gmx-developers mailing list