Жиза №621
Я тимлид в продуктовой команде, работаю с ребятами уже полгода. В целом команда нормальная, но есть у нас пара разрабов, которые считают, что их код — это произведение искусства, а мои замечания — личное оскорбление.
На прошлой неделе решили внедрить новый процесс код-ревью. Я собрал всех, объяснил, зачем это нужно, показал примеры. Все кивают, вроде согласны. На следующий день присылаю первому разработчику фидбек по его пулл-реквесту — пара предложений, ничего критичного. Через час он врывается в чат: — Ты серьезно? Это рабочий код, зачем его трогать? — Ну, он работает, но читается как древний свиток. Давай упростим. — Это мой стиль, мне так удобно.
Доказываю, что удобство для команды важнее личных предпочтений. Через полчаса споров он все-таки согласился, но скрипя зубами. Будто я ему должен теперь премию. На следующий день второй разраб выдал то же самое, только с добавкой: «Я вообще-то три года так пишу, и никто не жаловался».
К концу недели я уже чувствовал себя как дрессировщик в цирке. Но в пятницу на ретро один из них вдруг выдал: «Ладно, с код-ревью реально стало понятнее, что к чему». Остальные промолчали, но хотя бы не спорили. Прогресс? Прогресс. ______________ Больше историй в тг «Жиза ИТ руководителя»
· 11 ч
Да, позиция "Я художник, я так вижу" довольно смешна, когда речь идет об исходном коде. Надо принять стандарт кодирования (coding standard) и придерживаться его. Как говорится "У меня был некоторый план и я его придерживался".
ответить
коммент удалён
· 7 ч
Наверно, таким образом человек пытается показать свою исключительность и экспертность, и раз никто не хочет тратить время на разбор этой клинописи, то это доказывает некомпетентность всех вокруг, кроме автора😁
ответить
ответ удалён
· 6 ч
Это какая-то неадекватность. Наоборот, программист должен быть заинтересован, чтобы его код был понятным и прозрачным. Тогда люди им будут пользоваться и авторитет программиста будет расти.
ответить
ответ удалён