arrow

назад

ask

Вопрос

Должен ли team lead в разработке сам ревьюить код? Или это спустить на подсиненных, или вообще должна быть общая кросс-командная система ревью? Опишите свой опыт.

repost

220

input message

напишите коммент


8 комментов

Ревью можно поручить ИИ , но для начала надо подготовить качественный code rules.

0

ответить

По логике вещей, чем больше знающих людей посмотрят на твою работу, тем больше ошибок они выявят. Наверное, трех специалистов должно хватить, в лучшем случае, конечно. Team-lead, конечно, должен принимать участие, а иначе, в чем его лидерство?

0

ответить

На сколько знаю, в некоторых компаниях тимлид выполняет чисто менеджерский функционал: распределяет задачи, следит за сроками, общается с подчинёнными на тему их дальнейшего развития (повышения грейда). Видимо, в таких компаниях как-то по-другому выстроен ревью кода, между коллегами, например.

0

ответить

Это плохая практика. Начальник тоже должен брать на себя какую-то часть работы, а не только руками водить.

0

ответить

Обычно выстраивается зависимость между локализацией изменений и ответственными на данном участке. Привязывать ревью к позиции или роли - стрелять себе в ноги.

0

ответить

«Подсиненных» — это про пик Балмера?

0

ответить

это пьяненькие, но еще не сильно ))

0

ответить

Смотря кого вы подразумеваете под тимлидом. Я прошла путь от разработчика до тимлида кросс-функциональной команды и в процессе вывела для себя несколько правил.

Если команда смешанная, то тимлид может ревьить и только то, направление из которого вырос, но не обязан иначе времени на управленческий функционал может не остаться.

Если тимлид управляет какой-то конкретной ролью, например, только фронты, то это его обязанность и код писать на уровне синьора и код ревью проводить.

Кросс-командное ревью имеет право на жизнь, если команда от 2х человек в одной роли и в одном стеке. Потому что в каждом фреймворке своя специфика и при кросс код ревью разных стеков все будет сводиться к проверке базовых правил, частично решаемых линтером и частично здравым смыслом опытного разработчика 🙌

0

ответить

еще контент автора

пост закреплён — пока закрепить можно только один пост

trash bin
перейти к нему не получится