Что такое code review

Что проверяют

Есть ли у вас культура работы в команде и понимаете ли вы, что ревью это не поиск виноватых.

Как отвечать

Ревью кода это чтение чужих изменений до попадания в основную ветку. Цели три, и дефекты среди них не главная:

• распространение знаний: после ревью изменение понимает не один человек;
• единый стиль и архитектурная согласованность: чтобы код через год читался как написанный одним человеком;
• поиск проблем: логических ошибок, пропущенных краевых случаев, потенциальных утечек.

Что делает ревью работающим:

• маленькие изменения. Пул-реквест на 200 строк читают внимательно, на 2000 ставят «одобряю» не глядя. Это самый сильный фактор;
• автоматизация формального. Форматирование, линтер, типы и тесты проверяет машина в CI, чтобы люди обсуждали смысл, а не пробелы;
• описание от автора: что и зачем, как проверить, что осталось за рамками. Самопроверка перед отправкой снимает половину замечаний;
• быстрый ответ: ревью в течение дня, иначе автор уходит на другую задачу и возвращаться дорого;
• разделение важности замечаний: блокирующее отделять от вкусового, помечая последнее как необязательное;
• тон: обсуждается код, а не человек. Вопрос «а что будет, если сюда придёт пустой массив?» работает лучше указания «переделай».

Со стороны автора: не воспринимать замечания как претензии, отвечать на каждое, а спор длиннее трёх сообщений уносить в устный разговор.

Где спотыкаются

Присылают огромные изменения и получают формальное одобрение.

Спорят о стиле, который должен решать форматтер.

Пишут замечания в форме оценки человека.

Считают ревью этапом контроля качества и не проверяют код сами перед отправкой.