Settings and activity
7 results found
-
4 votes
Jesse Jacob
shared this idea
·
-
19 votesJesse Jacob
supported this idea
·
-
15 votesJesse Jacob
shared this idea
·
-
7 votesJesse Jacob
supported this idea
·
-
47 votes
An error occurred while saving the comment -
51 votes
An error occurred while saving the comment Jesse Jacob
commented
I think you're trying to make the tool patch over a bad code review process...I don't see many people needing this feature if they're doing frequent reviews. In my experience this problem crops up when people wait a very long to time to post a review a review--so long that other people can make unrelated changes to the same files as you've described. This is a pattern you should try to avoid by having people submit reviews more frequently so the changes being reviewed are more granular.
-
32 votesJesse Jacob
supported this idea
·
I believe this is a limitation of TFS itself. MS has said they will introduce an iterative pre-commit code review policy in TFS 2015 around the Update 1 timeframe. Hopefully ReviewAssistant will be able to support this work when it's available as they did with the existing pre-commit code review policy. I wouldn't hold your breath waiting for them to implement this on their own.