Ревью

Ревьювер - человек, который осматривает код.

Ревью - поверхностный осмотр кода. Поверхностным осмотр является потому, что ревьювер не несет ответственности за код.

Нюансы

• Чем больше количество разработчиков выполняют осмотр, тем больше надежд разработчик возлагает на ревьюверов, поэтому ревьювер всегда 1, максимум 2.
• Подходы осмотра - чем больше кода, тем больше замечаний к коду, а значит тем больше подходов осмотра. Но ревью стараемся проводить максимум в 2 итерации! Если ревью в 2 итерации не укладывается - разработчик с ревьюером устно пытаются совместно разобраться в проблеме.

Заметки

Замечено, что качество осмотра падает в следующих случаях:

1. ухудшение осмотра прогрессирует от первого к последнему подходу осмотра (самое тщательное ревью - первое, самое поверхностное - последнее), интересно то, что увеличение кол-ва строк кода никак не влияет (не в лучшую сторону, ни в худшую)
2. чем меньше времени ревьювер готов уделить на ревью, тем хуже осмотр (ревьювер может быть загружен своими задачами)

Источники: 1