Код ревью? Я сам всё проверю — так надёжнее

Признавайтесь, кто так думал?

Я был тимлидом в продуктовой команде из 8 разработчиков. И у меня был пунктик: каждый коммит перед мержем я должен посмотреть лично. Даже если задача плёвая - исправление опечатки в интерфейсе.

Команда сначала терпела. Потом начала раздражаться. А потом просто перестала выкладывать код до пятницы, потому что «всё равно Сергей Петрович завернёт».

Итог: ➡️Очередь из пулл-реквестов висела по 3 дня. ➡️Релизы выкатывали раз в две недели, хотя могли каждый день. ➡️Джуны не учились принимать решения - ждали моей резолюции. ➡️Я работал по 10 часов, проверяя чужой код вместо своих задач.

Команда не выгорала. Она закисала.

Переломный момент

Однажды сеньор написал в общем чате: «Сергей Петрович, может, нам доверять код-ревью друг другу? Мы же не дети».

Сначала я обиделся. Потом задумался.

А потом мы собрались и придумали новые правила.

Что сделали: ✅️Ввели правило двух плюсов. Мержить можно, если два разработчика (не я) посмотрели код и поставили апрув. Я в этой схеме - крайний случай. ✅️Прописали чек-лист ревью. Не абстрактное «всё переделать», а конкретно: архитектура, тесты, нейминг, безопасность. Сеньоры обучили джунов проходить по этому списку. ✅️Сняли ответственность за ошибки с одного человека. Если баг улетел в прод - разбираемся командой, а не ищем виноватого. ✅️Я перестал лезть в каждый PR. Вместо этого раз в неделю проводил ретроспективу по качеству кода.

❗️Что получили через 3 месяца ✅️Время от коммита до прода упало с 3 дней до 4 часов. ✅️Частота релизов выросла до 2–3 раз в неделю. ✅️Джуны перестали бояться ошибаться и начали предлагать архитектурные правки.

Я освободил 15 часов в неделю на развитие продукта и общение с командой. Текучка в команде обнулилась - за полгода никто не уволился.

И самое главное: исчезло напряжение. Люди перестали оглядываться на меня и начали дышать.

🤔Что я понял

Контроль не рождает качество. Он рождает страх и замедление.

Доверие - не про то, чтобы закрыть глаза и надеяться на лучшее. Доверие - это про: 🔹️чёткие правила, 🔹️разделённую ответственность, 🔹️и право на ошибку (с разбором полётов, а не с казнью).

Мы тогда написали одностраничное «Соглашение о командных правилах код-ревью». Оно до сих пор висит в нашем Notion и помогает новичкам быстро вливаться.

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

Код ревью? Я сам всё проверю — так надёжнее | Сетка — социальная сеть от hh.ru