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