По-хорошему надо комитить каждое логически отдельное изменение. Например, поправил баг - коммит, исправил опечатку - коммит, добавил новое поле - коммит.
Тогда слишком много коммитов получится
много для чего?
как будто что-то потом мешает их сосквошить
от команды зависит, кто-то просит коммитить каждый чих, кто-то по одному коммиту на задачу
О, спасибо за наводку
Коммит за задачу изврат какой-то.
Это действительно непривычно
а вот интересно - коммит на каждый чих тому, кто просит нужен зачем? минусы могу сообщить - при rebase с долгой историей конфликты придется на каждый коммит чинить, чтобы понять суть и объем изменений придется эти коммиты вместе собирать и смотреть. Но вот вопрос - коммиты делаются черти как. я понимаю, если в 1 PR решается несколько задач - тогда можно несколько коммитов. Но обычно 1 PR - одна задача - так гораздо проще поставлять изменения и ревьюверам смотреть на отдельные правки с четким минималистичным описанием, чем на пачку изменений с длинным описанием (вспомним еще, что на описания PR обычно есть лимиты по тексту - и в них можно просто не уложиться решая 100500 задач в одном PR)
не вижу противоречий, глобально, но никогда не встречал необходимости ни со стороны кодера, ни с ревьюверской видеть шаги, которыми достигалось решение одной задачи - это можно текстом в PR описать сразу в понятной форме. Чем смотреть "всякие поправил папочку 1", "поправил файл 2" в git истории
rebase -i HEAD~89 и поехал
Обсуждают сегодня