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