views:

5884

answers:

13

I want to be able to do a peer code review with Subversion, but what's the easiest way to do this without installing a server application or paying for an online service?

PS: This is a .Net project in Visual Studio.

+4  A: 

If you are using Eclipse, I recommend the Jupiter plugin.

scubabbl
edited to fix link.
scubabbl
Sorry, this is all .Net development in Visual Studio.
First time that I hear about Jupiter. Sure I will try it.
marcospereira
+2  A: 

If you are using TortoiseSVN, you should open the Log from the root of the branch you are reviewing. Select the revision in the top pane, and the bottom pane will list all the files that have changed. I haven't used it for a while, but you should be able to view the diffs of the changes on the files that changed in the revision.

RKitson
+1  A: 

Can the developers sit together? Can they do pair programming? If they can, you won't need a tool. :-)

Kind Regards

marcospereira
Still may wish to capture results of the review.
Brian Carlton
A: 

Use the TortoiseDiff, put it on a big screen with everybody sitting around the screen and start mapping changes to requirements. That will be great but if they are all together all the time, just make them (or you) do peer reviews frequently and no code review will be needed except that you want to get everybody on the same page.

+5  A: 

I actually figured out another easier way to do this.

Creating the Diff Pack

  1. Make the changes that you want to have reviewed.
  2. Goto the root of your project in Windows Explorer and right click, 1. Click Tortoise SVN -> Create Patch
  3. Select the files you want to appear in the review. Click OK
  4. Save your diff pack to a file name that describes the changes.

Viewing the Diff Pack Against Local Changes

  1. Copy the pack to the project root, if it's already there.
  2. Right click on the file 1. Click Tortoise SVN -> Apply Patch
  3. Double click the file that you want to view in the File Patches pane
  4. When finished, close the window to avoid applying the patch to your local store.

Viewing the Raw Changes

  1. Just double click the patch file.
A: 

Why you want to connect the review process with your IDE ?
It's sounds a little bit disconnected. Usually review system will be integrated with your source control and not IDE.
You can google for "code review tools" and will find a lot of options.
Unfortunately the one i can recommend you can't get :) it's home grown,
But i think that tools is a second stage,for the start you can do a code review in pair. Just using simple diff program before you committing your code to source control with a team leader/senior engineer will do the magic. You will see how many bugs you will prevent by simple code review without any tools.

Ilya
+4  A: 

You should set up trac, it has integration with subversion and has a peer review plugin on trac hacks. This will give you the ability to keep trac of code reviews easily without having to manually do diffs.

Also, if trac/peer review plugin is to much overhead for you... VMWare is developing a code review application that is completely open source here.

sontek
+2  A: 

I would say to look at this posting. I personally like Rietveld myself.

fuentesjr
A: 

I'll recommend 'CodeReviewer' from SmartBear Software.

It's not free, but I think it does its job quite well for its bucks.

http://www.smartbearsoftware.com/codecollab-codereviewer.php

all2one
A: 

Check out Codestriker -- it's free and open source. We've been using it for several years and it has great integration with svn.

A: 

The code review plugin for Trac is really nice. Actually the whole source code integration piece for Trac is nice. Check it out, you won't be disappointed.

Brian Hasden
A: 

What are the steps need to get Peer Review Icon in trac after successfully install? -- Ramesh

A: 

Check out my own visual stuio add-in I did for doing reviews within Visual Studio IDE

http://www.codeproject.com/KB/work/ReviewPal.aspx

Chathuranga Wijeratna