Руководитель группы разработки в Российский экспортный центр
· 11.06Вопрос
Должен ли team lead в разработке сам ревьюить код? Или это спустить на подсиненных, или вообще должна быть общая кросс-командная система ревью? Опишите свой опыт.
8 комментов
· 11.06
По логике вещей, чем больше знающих людей посмотрят на твою работу, тем больше ошибок они выявят. Наверное, трех специалистов должно хватить, в лучшем случае, конечно. Team-lead, конечно, должен принимать участие, а иначе, в чем его лидерство?
0
ответить
коммент удалён
· 17.06
На сколько знаю, в некоторых компаниях тимлид выполняет чисто менеджерский функционал: распределяет задачи, следит за сроками, общается с подчинёнными на тему их дальнейшего развития (повышения грейда). Видимо, в таких компаниях как-то по-другому выстроен ревью кода, между коллегами, например.
0
ответить
ответ удалён
· 17.06
Это плохая практика. Начальник тоже должен брать на себя какую-то часть работы, а не только руками водить.
0
ответить
ответ удалён
· 11.06
Обычно выстраивается зависимость между локализацией изменений и ответственными на данном участке. Привязывать ревью к позиции или роли - стрелять себе в ноги.
0
ответить
коммент удалён
· 11.06
«Подсиненных» — это про пик Балмера?
0
ответить
коммент удалён
· 13.06
это пьяненькие, но еще не сильно ))
0
ответить
ответ удалён
· 11.06
Смотря кого вы подразумеваете под тимлидом. Я прошла путь от разработчика до тимлида кросс-функциональной команды и в процессе вывела для себя несколько правил.
Если команда смешанная, то тимлид может ревьить и только то, направление из которого вырос, но не обязан иначе времени на управленческий функционал может не остаться.
Если тимлид управляет какой-то конкретной ролью, например, только фронты, то это его обязанность и код писать на уровне синьора и код ревью проводить.
Кросс-командное ревью имеет право на жизнь, если команда от 2х человек в одной роли и в одном стеке. Потому что в каждом фреймворке своя специфика и при кросс код ревью разных стеков все будет сводиться к проверке базовых правил, частично решаемых линтером и частично здравым смыслом опытного разработчика 🙌
0
ответить
коммент удалён
· 11.06
Ревью можно поручить ИИ , но для начала надо подготовить качественный code rules.
0
ответить
коммент скрыт — часть юзеров считает его токсичным или некорректным
коммент удалён