We are beginning to use the code review functionality built-in to VS 2012 and VS 2013 preview. Requesting the review and adding comments seem pretty straightforward. If someone adds comments requesting the code to be changed, then how does the requester make these changes and show them?
So the process would flow like this:
How does Person 1 now show these changes to Person 2? You can add comments and send them, but the files don't change. I'm assuming the files are from the changeset created when the original review was requested. Should Person 1 close this review, and request a second review?
Creating a second review doesn't seem optimal, because you lose the history of your conversation of why you were making the changes.
There are ton of websites showing how to use the basic functionality of the code reviews, but are there any sites that show the best practices? For instance, who should be clicking the check boxes next to files?
So the process would flow like this:
Here are the steps necessary to update the shelveset associated with the review.
I've included some screen shots as I find it helps to clarify things.
1) From the "Code Review" pane select the "view shelveset" link as shown here:
2) From the "Shelveset Details" pane highlight and copy the shelveset name as shown here:
3) Navigate to the "Pending Changes" pane, click on "Shelve" and paste the shelveset name for example:
4) Press the Yes button on the shelveset replace verification dialog: