how to mark code review defects in eclipse?

views:

224

answers:

+3 Q:

how to mark code review defects in eclipse?

i'm using eclispe 3.5 + mylyn + trac.

i'm looking for a way to mark source locations, that we identify in code reviews, in the IDE.

i need at least a way to collect all source locations and associate them with either a review or an assignment.

i'd prefer to use mylyn tasks as much as possible, but i can settle for the most basic eclipse use.

anyone have an idea?

thanks, asaf :-)

+2 A:

The tool closest to what you are looking for is still a proposal (to be reviewed tomorrow actually):

ReviewClipse

alt text

The only (open source? at least free?) code review tool I know of for Eclipse is currently developed
(part of a flash source editor Sourcemate)

Hopefully, it will be available as an independent product...

alt text

The other actor in that area is a major one: IBM Rational Software Analyzer Version, with integration with IBM Rational Team Concert, which runs on the Jazz platform.
Support also C/C++. Available in 7.1 version

alt text

VonC 2010-02-09 13:44:40

ReviewClipse requires subversion, which we don't use. since i'm looking for a free, independent tool, i can't use the others. do you have something of a "best practice"? thanks anyway

Asaf 2010-02-09 14:06:43

@Asaf: note, ReviewClipse supports Git and CVS in addition to the SVN connectors (http://www.eclipsecon.org/2010/sessions/sessions?id=1400). Other than that tool, I do not know about other code review related "best practices". To be sure, Check the connector list: http://tasktop.com/blog/eclipse/mylyn-connector-discovery

VonC 2010-02-09 14:27:48

@VonC: i'm using your best pal, ClearCase. oh, and no plans to buy any tasktop products soon. as for reviewClipse, its dependency on the changeset makes it dependent on version control systems, while i prefer a general tool that its data is contexed by mylyn.

Asaf 2010-02-09 14:44:44

@Asaf: ClearCase user? I share your pain ;)

VonC 2010-02-09 15:46:19

see http://stackoverflow.com/questions/1878506/best-free-code-review-tool-for-eclipse-java-flex-development/2235339#2235339

Asaf 2010-02-10 08:29:43

+2 A:

here is how we're gonna do it, pls share your opinion:

1. add a task tag (optional)

task tag

2. add a task to the context

add a task by double clicking it.
notice the Apply Context button on the view's top right toolbar.

alt text

3. review your code, while adding tasks to the mylyn task context

as for cleaning up the tasks, we haven't got there yet :) it seems the author fixes the issues, she deletes them. (which is a shame, all that work done, and no way to save it w/o cluttering the code)

asaf :-)

Asaf 2010-02-10 09:14:46

@Asaf good solution (+1), except the lack of history (once a task is done, it is deleted)

VonC 2010-02-10 09:24:30

ansaurus

tags:

views:

answers:

how to mark code review defects in eclipse?

1. add a task tag (optional)

2. add a task to the context

3. review your code, while adding tasks to the mylyn task context

related questions