LINUX.ORG.RU

История изменений

Исправление mix_mix, (текущая версия) :

Ты так и не понял зачем это делается. Вот есть задача что-нибудь заимплементить или пофиксить. При этом по умолчанию считается, что задача уже тривиальна ровно настолько, что нет смысла разбивать ее на подзадачи. Программист в отдельной ветке пишет код, отправляет на ревью и вносит правки в соответствии с комментариями коллег. Последние два шага повторяются до тех пор, пока все не будут довольны качеством кода и, собственно, решения. В конце все 20 (или сколько там) коммитов этой ветки сквошатся в один «Заимплементил Х» или «Пофиксил Y» и отправляются мерж реквестом. Смысл в том, чтобы не устраивать свинарник вида http://engineering.adslot.com/assets/branch-madness.png Одна задача – один коммит.

Исходная версия mix_mix, :

Ты так и не понял зачем это делается. Вот есть задача что-нибудь заимплементить или пофиксить. При этом по умолчанию считается, что задача уже тривиальна ровно настолько, что нет смысла разбивать ее на подзадачи. Программист в отдельной ветке пишет код, отправляет на ревью и вносит правки в соответствии с комментариями коллег. Последние два шага повторяются до тех пор, пока все не будут довольны качеством кода и, собственно, решения. В конце все 20 (или сколько там) коммитов этой ветки сквошатся в один «Заимплементил Х» или «Пофиксил Y» и отправляются мерж реквестом. Смысл в том, что нет желания устраивать в релиз ветках свинарник вида «а вот тут я переименовал переменную». Одна задача – один коммит.