​Шаблон для пулл реквестов В проектах, где я работал, обычно | Менеджер от боженьки

​Шаблон для пулл реквестов

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

У ребят из Soundcloud процессы на высоте, они придумали целую систему и заполняют:
описание задачи
детали реализации (тут порефакторили, там добавили новое поле в базу)
скриншот до\после
как тестировать

Это очень хорошие вопросы, о которых надо позаботиться заранее. Без шагов по тестированию QA тратят кучу времени на разбор задачи. А без деталей реализации тимлид долго въезжает в код ревью.

С таким подходом ПРы становятся более предсказуемыми и команда закрывает задачи быстрее, снижая cycle time.

Самое крутое, что есть настройка, чтобы ПР автоматически предзаполнялись этими полями. То есть нажал "открыть ПР" и эти заголовки уже тут, осталось только внести контент.

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