Search
Write a publication
Pull to refresh

Comments 3

Могу порекомендовать также следующее. Очень помогает, когда автор кода оставляет уже в самом пул реквесте комментарии на свой же код. Пара строк может быстро раскрыть необходимость какого-нибудь не особо эстетичного костыля и заметно упростить жизнь ревьюера. Ну и к этим комментариям также зачастую могут вернуться исследователи кода уже в будущем.

Не устаю повторять, что все "проблемы" с код-ревью начинаются с корявого процесса постановки задач. Если у вас в команде всем окей сначала ставить задачу "Нужно сделать хорошо", а потом делать коммиты на 2к строк в 90 файлах, которые меняют все и сразу, то да, код-ревью будет гемором. А потом тестирование будет гемором. А потом выкатка в прод. А потом сопровождение.

Поэтому я всегда начинаю онбординг новичков с пояснения принципа декомпозиции задач. И спрашиваю за этот принцип гораздо строже, чем за всякие кодстайлы.

коммиты на 2к строк в 90 файлах
Ну, ради объективности, это бывает и независимо от кривой постановки задачи. В смысле, хорошая постановка не обеспечивает хорошей декомпозиции — в конце концов, исполнитель тоже должен о декомпозиции думать.
Sign up to leave a comment.