Что плохого в количестве коммитов чуть больше, чем за которое могла решиться задача на самом деле?

Question

[Twitt]

Устроился на новую работу, поставили задачу, сделал в ветке один коммит, на ревью сказали что можно порешать немного по другому. По итогу еще 4 коммита сделал, ревьюер говорит: "там уже 5 коммитов, давай новый PR".
В чем проблема 5 коммитов, может кто то может подсказать?

Comments

[Andrej Sharapov]

придирки. я делаю сайты и вношу правки по сто раз, на след день делаю их снова, в поисках "лучшего решения". Не вешать же меня за это. Ну да, был бы профессиональнее, делал бы меньше, но пока вот так.

[Дмитрий Арушанов]

Вроде как, все изменений (остальных 4 коммитов) и так подтянутся в PR если он создан. Если комиты были в туже ветку.

Может просто проблема в том, что пока вы делали, свои 4 коммита, в репозиторий были внесены изменения другими участниками? И теперь PR проблемный?

Ну или может хотят что бы вы склеили комиты в 1 общий?

[Kovalsky]

Никто вам ничего конкретного тут не скажет, у всех свои стандарты разработки. Кто-то делит комиты по одному признаку, кто-то - по другому. Отвечая на вопрос "в чем проблема 5 коммитов" - технически естественно никакой проблемы нет, но если на вашей новой работе предпочитают в комит класть все связанные с фичей или фиксом изменения в один комит, то естественно 5 комитов это вообще ад для одной задачи.

Answer 1

[Антон Швец]

В личных тараканах ревьюера. Если уж его так парят коммиты при слиянии PR можно слепить коммиты в один.

Наоборот, чем меньше изменений в коммите. тем лучше. Я вот страдаю обратным, в конце дня делаешь коммит, потом не разберешься во всем этом.

Comments

[Антон Швец]

Иван, ну так у автора вопроса так и есть. Своя ветка, в ней коммиты. Прочитайте вопрос то.
Похоже просто что PR был уже принят, поэтому понадобился новый просто.

Answer 2

[Северное Сияние]

в идеальном мире, где все какают бабочками, коммит должен быть законченной фазой какой-либо правки, подразумевающей, что закончена конкретная (под)задача и в систему внесена правка, не требующая доработок и гарантирующая работоспособность системы.

в реальной работе всё бывает по-другому.

твои 5 коммитов - от волнения.

по сути, надо было просто все проверить и сделать еще один коммит.

зачем там PR мне не ясно