[Cosmo-dev] Bug fixes and unit tests

[Jeffrey Harris]

Not that I have a say in Cosmo team processes, but I'll say that in the
past I found the bugzilla review process tedious, so I wrote subzilla so
I could fire off one shell command to post a patch to a bug with a
review request.  Since then I haven't found reviews tedious. :)

Sincerely,
Jeffrey

Brian Moseley wrote:
> On 9/28/06, Ted Leung <twl at osafoundation.org> wrote:
> 
>> Also,  when Chandler goes into feature freeze, all checkins must be
>> code-reviewed before they can be committed -- patches containing the
>> fixes are attached to bugzilla bugs and the bugzilla mechanisms are
>> used to review the patches.    I think that this is a very helpful
>> process and wanted to know what people thought about adopting this idea.
> 
> i've never been a fan of this practice. it's too heavyweight. i for
> one review every svn commit message, and i squawk when i see something
> i don't understand or think could be done better. our team is still
> small enough that i think this practice will keep enough eyeballs on
> the code without going through the tedious bugzilla review process.
> _______________________________________________
> cosmo-dev mailing list
> cosmo-dev at lists.osafoundation.org
> http://lists.osafoundation.org/mailman/listinfo/cosmo-dev