5 правил пулл-реквестов

Решает одну задачу

Если рефакторите код или тесты — это отдельный пулл-реквест.

Правильно решенная задача с большим количество кода выглядит вот так:

Не больше 500 строк

Если пишете много кода — сливайте по чуть-чуть. Не пишите огромное непонятное

говно: его все равно никто вдумчиво не посмотрит.

1
 Состоит из унитарных коммитов

Один коммит на 300 строк — плохо, его никто (в том числе и вы через месяц) не
поймет. Как делать хорошие коммиты написано тут.

Хорошая история выглядит так:

Привязан к задаче

Клево, когда любому заходящему в ПР понятно, зачем вы, собственно, все это
сделали.

2
 Называется по-русски

Ваши 840/manager-fix не говорят никому ничего. А «Пофиксил вход для менеджеров

первой линии» — вполне.