Замечания к чужому коду

Замечания к чужому коду

Проверка кода ссорит команды чаще, чем споры о технологиях, и обычно из-за формы.

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

Пишите про код, а не про человека.

Объясняйте причину: замечание без объяснения — приказ, с объяснением — обучение.

Больше десяти правок — зовите на разговор, а не пишите простыню.

Отмечайте хорошее.

И держите темп: код, висящий на проверке три дня, устаревает. В распределённой команде это особенно больно — при разнице во времени каждый круг замечаний стоит целых суток, поэтому замечания собирают в один заход, а не выдают порциями.

И договоритесь в команде о сроке ответа на проверку: сутки, полтора — но чтобы срок был назван. Ожидание без срока раздражает сильнее, чем сами замечания.

IT Якутск — t.me/it_yakutsk