[Cosmo-dev] Bug fixes and unit tests

[Brian Moseley]

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.