[p4] Simple Code Review Tool

[Paul Goffin]

It's a Big Mistake to do this.

If you put barriers between your SCM system and your developers
they'll stop using it and it wont be long before you have code out
in the world that isn't in your system but is on a developers
workstation waiting to be checked in when he "gets round to it".

Let your developers submit code freely to a development branch
and then run your review tool before the branch is integrated into
the mainline.

Paul.




-----Original Message-----
Sent: 28 July 2007 01:56
To: perforce-user at perforce.com
Subject: [p4] Simple Code Review Tool


Hi there,

Has anyone created a simple code review tool for Perforce
that allows developers to review code changes before they
are submitted to the depot? One of my developers is
working on a simple email that would be sent to a
reviewer containing the output of the p4diff command.
However he is running into a problem figuring out how to
handle new files that are being added as part of the
pending changelist. Any insights would be helpful?

Thanks,

-Scott

Scott Marshall | Build Engineer


No virus found in this outgoing message.
Checked by AVG Free Edition. 
Version: 7.5.476 / Virus Database: 269.10.22/923 - Release Date: 27/07/2007
18:01