Ответ вслух Senior Код-ревью обновлено

Как ускорить ревью в команде

Вопрос

В команде PR висят на ревью по три дня. Как бы ты это исправил?

Попробуйте сами

Ответьте вслух за 2 минуты, как на собесе. Потом сверьтесь с тезисами: что прозвучало, а что нет.

Показать ответ и разбор

Коротко

Сначала измерю: время до первого ответа, время до мержа и размер PR, обычно корень в огромных пулреквестах и одном перегруженном ревьюере. Дальше договорённости: PR до нескольких сотен строк, крупное режем на стек зависимых PR, шаблон описания с тем, как проверить.

Что должно прозвучать в ответе

  • Измерить: время до первого ревью и до мержа, размер PR

  • Маленькие PR и стеки PR, шаблон описания

  • Автоматизация в CI: линтер, типы, тесты, превью-стенды

  • Договорённость о сроке реакции и слоты для ревью в дне

  • CODEOWNERS и распределение ревьюеров, чтобы не было узкого горлышка

  • Парное программирование для сложных задач

Что могут спросить следом

  • Как убедить команду писать маленькие PR?
  • Что такое stacked PR?

Разбор

Сначала измерю: время до первого ответа, время до мержа и размер PR, обычно корень в огромных пулреквестах и одном перегруженном ревьюере. Дальше договорённости: PR до нескольких сотен строк, крупное режем на стек зависимых PR, шаблон описания с тем, как проверить. Всё механическое уходит в CI: линтер, типы, тесты, превью-стенд для UI, чтобы человек смотрел только смысл. Договариваемся о сроке первой реакции, например в течение рабочего дня, и выделяем слоты на ревью. CODEOWNERS и автоматическое назначение распределяют нагрузку. Для сложных задач полезно парное программирование: ревью происходит прямо во время работы. Через месяц снова смотрю на метрики.

Ответьте сами, а не прочитайте

В тренажёре вопрос попадёт в повторения и вернётся, когда начнёт забываться. Плюс билеты с зачётом: 20 вопросов, не больше 2 ошибок.