- Before introducing code reviews in your project, agree on a procedure for how to do it, and make sure that everyone agrees with it and understands it. I can suggest the following procedure points, just to get started:
- Review Assistant tool includes the custom check-in policy for TFS
- setup a custom policy to TFS that every check-in needs to pass a code review
- FishEye
Plan for a maximum of four hours development time
Keep the master branch as the production ready branch at all times
Always branch out a new task from master or release
Use a standard for naming your task branches (more on this below)
Use task branches for every change (no exception!)
When done coding, push to
Do a code review of your own pull request before sending it
Never merge to master until you get at least one "Ok" comment from others
Check others pull requests at least every two hours
http://share.ez.no/blogs/arne-bakkebo/branching-and-code-review-procedures-qa-4
Team Foundation Server provides
https://www.devart.com/review-assistant/docs/index.html?adding_code_review_policy_to_tfs_project.html
So, instead of developers having to shelve their changes manually and assign the
http://blog.devart.com/creating-tfs-custom-check-in-policy-part-1.html
FishEye knows everything about your code: search
Keep a pulse on everything about your code:
Visualize and report on activity, integrate source with JIRA issues, and search for commits, files, revisions, or people.
http://www.atlassian.com/software/fisheye/overview
- stash
https://www.atlassian.com/software/stash
- Gerrit
Gerrit is a free, web-based team software code review tool.
Software developers in a team can review each other's modifications on their source code using a Web browser and approve or reject those changes.
It integrates closely with git, a distributed version control system.
https://en.wikipedia.org/wiki/Gerrit_(software)
Gerrit
Gerrit is a
https://code.google.com/p/gerrit/
- A code review process is a process in which
a change by a developer by other developers.is reviewed
Every developer can suggest changes and update the suggested changes.
Once the change
While
Gerrit is a code review system developed for the Git version control system
Advantages of code review
Early error detection: errors are early identified in the process
Conformity with coding standards: code review allows the team to identify early in the process any violations with the code standards of the team
Knowledge exchange: the code review process allows new developers to see the code of other developers and to get early feedback on their suggested changes
Shared code ownership: by reviewing
Code reviews in open tools provide people without the permission to push to a repository with a simple way to contribute their suggested changes and to get feedback.
http://www.vogella.com/articles/Gerrit/article.html
- Before you can commit the files to your repository, you need to add them.
Simply right click the shared project’s node and navigate to Team =>Add
After this operation, the question mark should change to a plus symbol. To set certain folders or files to
After changing files in your project, a “>” sign will appear right after the icon, telling you the status of these files is dirty
http://eclipsesource.com/blogs/tutorials/egit-tutorial/
- Git blame
git status git tag git blame
The git blame command is a versatile troubleshooting utility that has extensive usage options. The high-level function of git blame is the display of author metadata attached to specific committed lines in a file. This is used to examine specific points of a file's history and get context as to who the last author was that modified the line. This is used to explore the history of specific code and answer questions about what, how, and why the code was added to a repository.
Git Blame vs Git Log
While git blame displays the last author that
The git blame command is used to examine the contents of a file line by line and see when each line was last
Online Git hosting solutions like Bitbucket offer blame views, which offer a superior user experience to command line git blame usage.
https://www.atlassian.com/git/tutorials/inspecting-a-repository/git-blame