[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Can Subversion aid code reviews?

From: Bicking, David (HHoldings, IT) <David.Bicking_at_thehartford.com>
Date: 2007-11-02 17:25:37 CET

We're looking at code review processes here, and I would like to know
what people here in this forum do for code reviews. Can Subversion
itself supply help in a code review process? Some suggest that
"promotion groups" are the way to go, which Subversion does not support.
A basic process we have in mind goes something like this:

1. developer is assigned a task, builds the code, unit tests it.
2. The team lead reviews the code, and might assign work to make
changes.
3. The developer makes the changes and notifies the team lead
4. The final review is done and the task is closed.

Some core issues here around source control include, how to identify
what falls in the task (which will span several revisions), how to
report on code that is unreviewed, and how to flag said code as
reviewed.

This sounds to me like "branch by task", but we're not necessarily
talking about large tasks, and we don't want to branch frequently (let
alone have each team member branch for each of their tasks).
 
Suggestions?

 

--
David 
*************************************************************************
This communication, including attachments, is
for the exclusive use of addressee and may contain proprietary,
confidential and/or privileged information.  If you are not the intended
recipient, any use, copying, disclosure, dissemination or distribution is
strictly prohibited.  If you are not the intended recipient, please notify
the sender immediately by return e-mail, delete this communication and
destroy all copies.
*************************************************************************
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@subversion.tigris.org
For additional commands, e-mail: users-help@subversion.tigris.org
Received on Fri Nov 2 17:26:01 2007

This is an archived mail posted to the Subversion Users mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.