Twitt
@Twitt

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

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

Наоборот, чем меньше изменений в коммите. тем лучше. Я вот страдаю обратным, в конце дня делаешь коммит, потом не разберешься во всем этом.
Ответ написан
php666
@php666
PHP-макака
в идеальном мире, где все какают бабочками, коммит должен быть законченной фазой какой-либо правки, подразумевающей, что закончена конкретная (под)задача и в систему внесена правка, не требующая доработок и гарантирующая работоспособность системы.

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

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

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

зачем там PR мне не ясно
Ответ написан
BojackHorseman
@BojackHorseman
...в творческом отпуске...
мэй би тут дело в отношении к работе? недопиленная задача уходит на ревью?
Ответ написан
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы