Re: [Codestriker-user] Feature request

[David S. <si...@us...>]

> - Now that both tasks have been completed, both developers would like to
> have each of their individual work reviewed.

This is the problem.  Codestriker is designed to be a "pre-commit" step.  

The usual scenario with development teams is that unreviewed code is _not_ 
allowed to be committed into the CVS repository.  Once code has been 
reviewed, then the developer is free to commit it in.

The motivation for this is that a bad commit can impact the whole development 
team, resulting in potentially a lot of wasted time for bad commits.

In your scenario, there should be no commits by developer 1 or 2.  Once they 
have finished their "work", then they produce a "cvs diff" which is then 
reviewed in Codestriker.

There is no problem with both reviewers doing this in parallel - CVS is 
designed to handle this merges and so forth.  

The challenge is to define sensible units of work, which aren't too large to 
review, but software teams become very experienced at doing this very quickly.

Cheers,
David