Как оставлять замечания к чужой работе

Как оставлять замечания к чужой работе

Ревью — место, где рабочий разговор чаще всего превращается в личный. Обычно из-за формы, а не из-за содержания.

Разделяйте обязательное и вкусовое явно: «здесь потеряется ошибка» и «я бы назвал иначе» — замечания разного веса. Когда они идут вперемешку, автор перестаёт понимать, что важно, и начинает спорить со всем подряд.

Объясняйте причину: «так не делают» не учит ничему, «при пустом списке здесь упадёт» — учит.

Спрашивайте, когда не понимаете, вместо того чтобы предполагать худшее: у странного решения часто есть причина, которой вы не знаете.

Не вываливайте тридцать замечаний разом: после десятого включается защита. Много правок — пятнадцать минут голосом решают больше, чем длинная переписка.

И помните цель: чтобы код стал лучше и чтобы автор в следующий раз сделал так сам.

IT Пермь — t.me/itnews_perm