Комментарии 5
Реальные кейсы давай
Если бы все так давали код ревью в реальной жизни, то цены бы им не было. Очень часто просто забивают на это и код пишется как пишется
если 2 дня никто не ревьюит — PR имеет право быть влит, если нет явных возражений.
Отличный способ сачкануть! Два дня игноришь уведомления - и свободен. А через месяц, когда всё сломается, уже никто не вспомнит, что ты был ревьюером, и пойдут бить по шапке автору, а ты и не при делах как бы.
О да, длинные перепалки в комментах PR это прям боль. Особенно в комплекте с подходом "комментарий просто чтобы докопаться". Очень токсичная история и очень демотивирует команду, особенно автора PR, очень важно такое останавливать и переключать на голос.
Ревью на статью о ревью, хэхэ
Задавай вопросы, а не выноси вердикты
Постоянный вопросы не панацея тоже, иногда проще "пальцем ткнуть и привести аргумент" особенно если это не какой-то принципиальный участок. Чрезмерные вопросы не меньше иногда вызывают раздражение и могут читать по разному.
И вот что важно: тимлид должен быть доступен. Если ревью ждёт из‑за того, что лид ушёл на стратегическую сессию на 2 дня
Лид такой же человек, а не машина по решению всех вопросов и всегда. Если команда вообще не умеет договорится, зная что Лида нет два дня и сидит надув губы, ну очевидно есть проблема в настроениях команды, а не в доступности Лида.
Установите таймбокс: если 2 дня никто не ревьюит — PR имеет право быть влит, если нет явных возражений.
Ну тут уже писали, что это треш. PR без ревью и апрува - просто лишние клацанье кнопок в гитлабе/гитхабе. При таком застое лучше присылать пуш Лиду и ревьюверу: у нас висяк, посмотрите.
А в остальном статье понравилась, она может и не приносит чего-то откровенно нового, но мне нравится иногда возвращаться из рутины к таким текстам, чтобы скорректировать курс, если где-то пошел крен.
Коротко о том, как внедрить код-ревью, которое работает (а не бюрократию)