Difference between revisions of "Code Review"
From Agility
m |
|||
| (One intermediate revision by one other user not shown) | |||
| Line 2: | Line 2: | ||
Code review is a process designed to: | Code review is a process designed to: | ||
| − | * | + | * improve code quality |
* catch possible bugs | * catch possible bugs | ||
* find logical mistakes | * find logical mistakes | ||
| Line 12: | Line 12: | ||
Main goals: | Main goals: | ||
| − | # | + | # Do not focus on code styling. |
# Foster a positive code review culture | # Foster a positive code review culture | ||
| − | # | + | # Assume that author had a good intentions while he was writing the code. |
| − | # | + | # Ask other persons to solve any resolution conflict. |
# Contact with author as a replacement of source code annotations. | # Contact with author as a replacement of source code annotations. | ||
| − | # Lines of code under review should be less than | + | # Lines of code under review should be less than 400 |
# Inspection rates should under 500 lines of code per hour | # Inspection rates should under 500 lines of code per hour | ||
# Do not review for more than 60 minutes at a time | # Do not review for more than 60 minutes at a time | ||
Latest revision as of 13:03, 6 September 2017
Code review is a process designed to:
- improve code quality
- catch possible bugs
- find logical mistakes
- check for performance issues
- ensure code is readable and maintainable
- spread knowledge of the code base throughout the team
- get new people up to speed with the ways of working
- expose everyone to different approaches
Main goals:
- Do not focus on code styling.
- Foster a positive code review culture
- Assume that author had a good intentions while he was writing the code.
- Ask other persons to solve any resolution conflict.
- Contact with author as a replacement of source code annotations.
- Lines of code under review should be less than 400
- Inspection rates should under 500 lines of code per hour
- Do not review for more than 60 minutes at a time
