Зачем делать ревью кода?
- Провека соответствия политикам и лучшим практикам – не все еще автоматизировано (и вряд ли возможно)
- Расширение кругозора в коде проекта – что как работает, что недавно изменили
- Расширение общего кругозора – как другие пишут – особенно для новичков
- Найти ошибки (как в постановке задачи, так и в реализации) – свежий взгляд
Поиск ошибок на последнем месте не случайно – это именно свежий взгляд, а не пристальный поиск ошибок. Ревьюер за ошибки не отвечает, а вот за несоответствие кода политикам и практикам – как раз отвечает.
Как делать ревью кода?
- Оповещение в каком-нибудь чате разработчиков, что MR можно смотреть. Можно довольнительно ревьюеров назначать через Git UI, а можно нет – как договоритесь.
- Ожидается, что где-то в течении 10 минут начнут смореть. Нормально до часа в зависимости от текущей нагрузки.
- Адекватно, если в MR меньше 50 файлов – иначе как ни крути качество ревью упадет.
- Обычно ревью занимает от минуты до 10 минут. Не ожидается, что ревьюер будет вчитываться в каждую запятую.