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