Чем меньше пулреквест, тем лучше: легко отревьюить, легко задеплоить, легко искать ошибки. Чем больше пулреквест, тем сильнее дизмораль: увидел +700 -60 в Гитхабе и приуныл.
Кроме того, маленькие пулреквесты дают параллелить задачу: собрали письмо-заглашку, показали дизайнерам, пока дизайнеры смотрят — делаем отправку этого письма. Один пулреквест — одна атомарная задача.
Конечно, бывают сложные ситуации. Например, апгрейд Рельс. Одна огромная задача с кучей изменений. Но даже тут можно обойтись крошечными пулреквестами:
Открываем ветку rails-6-upgrade.
Делаем маленькие пулреквесты не к master, а к rails-6-upgrade.
Ревьюим, мержим их в rails-6-upgrade.
Когда апгрейд готов, мержим rails-6-upgrade в master.
Top comments (0)