Как оставлять замечания, чтобы их принимали

Как оставлять замечания, чтобы их принимали

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

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

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

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

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

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

В маленькой команде, где вы работаете с этими людьми годами, тон обходится дороже содержания.

IT Владимир — t.me/itnews_vladimir