Замечания, которые принимают
Ревью — место, где технический разговор чаще всего превращается в личный. Обычно из-за формы.
Разделяйте обязательное и вкусовое явно: «здесь потеряется ошибка» и «я бы назвал иначе» — разного веса. Вперемешку они превращаются в шум, и автор начинает спорить со всем подряд.
Объясняйте причину: «так не делают» не учит ничему, «при пустом списке упадёт» — учит.
Спрашивайте, когда не понимаете, вместо того чтобы предполагать худшее.
Не вываливайте тридцать замечаний разом: после десятого включается защита. Много правок — пятнадцать минут голосом.
И держите в голове цель: чтобы код стал лучше и чтобы автор в следующий раз сделал так сам. Не чтобы вы показали, как сделали бы вы.
IT Новосибирск — t.me/it_novosibirsk
IT Новосибирск
@it_novosibirsk
Замечания, которые принимают
Этот пост опубликован в Telegram-канале IT Новосибирск. Подписаться можно по ссылке: @it_novosibirsk.