Comments 3
Могу порекомендовать также следующее. Очень помогает, когда автор кода оставляет уже в самом пул реквесте комментарии на свой же код. Пара строк может быстро раскрыть необходимость какого-нибудь не особо эстетичного костыля и заметно упростить жизнь ревьюера. Ну и к этим комментариям также зачастую могут вернуться исследователи кода уже в будущем.
Не устаю повторять, что все "проблемы" с код-ревью начинаются с корявого процесса постановки задач. Если у вас в команде всем окей сначала ставить задачу "Нужно сделать хорошо", а потом делать коммиты на 2к строк в 90 файлах, которые меняют все и сразу, то да, код-ревью будет гемором. А потом тестирование будет гемором. А потом выкатка в прод. А потом сопровождение.
Поэтому я всегда начинаю онбординг новичков с пояснения принципа декомпозиции задач. И спрашиваю за этот принцип гораздо строже, чем за всякие кодстайлы.
Без рутины и тоски: как провести код-ревью