Делюсь, может, кому пригодится. Команда шесть человек, два сервиса на Rails 7.1. Осенью средний PR был под 900 строк, ревью висело по три дня, и смотрели его, честно говоря, по диагонали. В феврале договорились:
- PR до 400 строк без учёта миграций, schema.rb и сгенерированных файлов;
- если больше, автор пишет в описании, почему нельзя разбить;
- первый ответ на ревью в течение рабочего дня;
- большая фича идёт в main частями за флагом, а не одной веткой на три недели.
За месяц средний PR стал 210 строк, от открытия до мержа в среднем 7 часов вместо двух с половиной дней. Минусы тоже есть: PR стало больше, и пара человек жалуется, что весь день прыгает между ревью. И не всегда понятно, как резать изменение схемы, чтобы каждый кусок был рабочим.
Цифры хорошие. Изменение схемы обычно режется на три PR: первый добавляет колонку и начинает писать в обе, второй переносит старые данные, третий переключает чтение и удаляет старое. Каждый выкатывается отдельно и откатывается без боли.
В Москве на прошлой работе был лимит 300, прижился. От прыжков спасало смотреть PR два раза в день, а не по каждому уведомлению
Егор, это просится в доклад на 10 минут, особенно цифры до и после. В апреле снова собираемся в коворкинге у Василеостровской, мосты к тому времени, может, уже начнут разводить, так что кто поедет с Васильевского на такси, лучше уходить пораньше. Записать?