Settings and activity
4 results found
-
7 votes
An error occurred while saving the comment 
James
supported this idea
·
-
35 votes
An error occurred while saving the comment James
commented
This would be amazingly useful to help younger/less-experienced team members identify areas where they can more easily improve (the so-called "low hanging fruit"), as well as identify areas that are common among several developers, and may point to a company culture issue.
James
supported this idea
·
-
3 votes
An error occurred while saving the comment James
commented
Agreed. Having the expected items in the review panel helps remove the "oh I forgot" responses.
James
supported this idea
·
-
32 votes
An error occurred while saving the comment James
commented
This is an absolute must for project management. In the TFS tools, I want to see work-items or some sort of link that links to the review, or at least indicates whether a review was completed.
The pre-commit rule is nice, but it can be overriden, and the only way to see this is to review the commit history manually.
James
supported this idea
·
I would add that this list should allow exceptions; e.g.: typically, we don't need to review the 'packages.config' file, but a particular developer is new, or is unlikely to truly need to modify the file.