Search Wiki:

New Business Value From Code Review

TeamReview exploits the advantages of Team System and the Vs.net extensibility model to reduce transition waste and surface new value from the code review process. Using Work Items to facilitate the code review process creates a completely in-IDE code review experience, the ability to project-manage the code review tasks, and the opportunity to get new value from the code review process through code review forensics.
0202%20Microsoft%20Team%20System%20TeamReview_thumb.jpg

Poster created by Willy-Peter Schaub, VSTS MVP and hosted by the Developer Readiness Program community server and the South Africa Architect community server

TeamReview Advantages

  • No/less code review meetings necessary
  • Same place/Same time obstacles are overcome
  • Makes code review analytics/forensics possible
  • Easier to split up code reviews both over time and to many people
  • Accountability - can easily determine if any review feedback was submitted and if that feedback was implemented
  • Ability to create predefined in-IDE code walk-throughs by any developer on your project without requiring your attendance

(top)

TeamReview by Example


Jane Coder submits a request to review code by entering a new Task and assigning it to Joe CodeReviewer

Code Review Request.jpg


Then in the IDE Joe CodeReviewer right clicks and uses the TeamReview menu

TeamReviewMenu.jpg


In the New Code Review Response form Joe selects a Team Query to find Jane's request and then fills out the information and assigns the new Code Review Response to Jane. Note here that the Title is pre-filled by TeamReview

NewCodeReviewResponse.jpg


Joe finds another contentious point and uses TeamReview again. This time TeamReview pre-populates the "Query From", "Associate With", and "Assign To" values using the information from the last time.

SecondCodeReviewResponse.jpg


When Joe is done submitting feedback he re-assigns the original review request from Jane back to her (not shown). Now the review is over and the original code author, Jane, wants to see what feedback was given. She uses the "Code Review Replay" option of TeamReview

ReplayMenu.jpg


The Code Review Replay window comes up and Jane chooses a Team Query to identify her original review request.

ReplaySelectReview.jpg


Jane starts replaying Joe's feedback by selecting each row in the grid one-by-one, and as she does TeamReview re-opens and re-highlights the section of code Joe originally noted as having an issue.

ReplayOutcome.jpg


Jane fixes the code from Joe's feedback and then wants to close the Code Review Response, so she double-clicks in the Code Review Replay Window to open the Work Item.

DoubleClick.jpg


Of course all of this works because TFS is being used in the background. Here's what Jane's original request looks like with the linked Code Review Response items.

LinkedItems.jpg


And here is what the Code Details tab of one of those items, Joe's feedback, looks like - with all the code heuristic details available for querying and reporting by TFS or SQL Reporting Services.

CodeReviewResponseWorkItem.jpg

(top)
Last edited Sep 1 at 10:10 PM  by JBBrown, version 33
Comments
cege7480 wrote  Apr 15 at 3:04 PM  
I haven't downloaded this yet... but wow... Well thought man. Well though and friggin' well done.

hinshelmw wrote  Apr 16 at 8:55 PM  
Fantastic... Will definatly give it a try!

kevinidzi wrote  May 21 at 6:31 PM  
What great integration! Great idea and the way it integrates with the tfs code review workflow is a great plus.

Updating...