Как правильно коммитить?

Question

[HelpSophie]

Вроде разумно делать коммиты так: сделала какой-то завершенный модуль/блок/действие, проверила - закоммитила. Ну или считаю, что сделала, не важно, какая-то завершенная логическая итерация.
Но при работе на нескольких компах, например, дома и на работе, появляется необходимость пушить в нелогичный (с точки зрения кода) момент. И появляются коммиты типа "поработала на ноуте".

Как правильно поступать, как не плодить такие плохие коммиты или как их потом удалять?

Answer 1

[Сергей delphinpro]

Делаем временную ветку temp1 и пушим туда все что угодно.
Потом, когда закончим пилить фичу, переключаемся обратно на master или dev и делаем
git merge --squash temp1
Все изменения перекочуют в master/dev, но не будут закоммичены.
Теперь можно делать обычный коммит с описанием фичи, все будет в одном коммите.
После всего удаляем временную ветку с ключом --force

Временную ветку лучше назвать как-то более понятно, чем temp1, если работаете не одна. feat-super-puper.

Comments

[l4m3r]

Гемор какой-то. Не показывает рабочий процесс.
В итоге один огромный коммит на целый таск?
А если потом поправить надо, заново временную ветку?

Создал topic branch и хоть 1000 коммитов туда залей.
Номер таска + пара слов "что пилил" в каждом.
Слился в dev/master потом и всё.

Временные ветки какие-то...

[HelpSophie]

l4m3r, Почему огромный коммит? Какого размера "кластер" фичи, такой и коммит получится.

Сергей delphinpro, спасибо, такая идея мне нравится

[Сергей delphinpro]

l4m3r, речь о личной ветке конкретного девелопера.
Пусть в проект добавляется новая фича "А", обладающая обширной функциональностью. Тимлид заводит новую ветку feat-a от ветки dev (та самая topic branch/feature branch). Программист пишет код. Рабочий день кончился, код не дописан, строка кода даже оборвана наполовину =). Девелопер делает себе временную ветку, сохраняет туда. Завтра он решит поработать из дома, пульнет себе эту ветку на домашний комп и продолжит работу. Когда он завершит то что делал, он сквошит все свои наработки из личной ветки в ветку feat-a под одним осмысленным коммитом "Запилил foo для фичи А".

Answer 2

[l4m3r]

Не стоит относится к гиту как к священной корове. Ничего не надо удалять. Отправили кривое сообщение - фиг с ним.
Не надо писать "поработала на ноуте". Пишите в духе "Добавляю X в модуль Y (часть 1)".

Comments

[HelpSophie]

не феншуйно как-то

Answer 3

[Андрей]

Как вариант - пушить во временный бранч.
А потом сделать мердж в основной бранч фичи и
git reset --mixed на последний нормальный коммит. И удалить временный. Тогда и следов не останется.

А если контора не против, можете вообще отдельный удалённый репозиторий для этих целей создать

А ещё можно посмотреть, как git хранит stash'ы, может, их можно копипастом переносить

Answer 4

[egor_nullptr]

Как правильно поступать, как не плодить такие плохие коммиты или как их потом удалять?

* пушим в одну ветку все плохие и хорошие коммиты
* когда работа над фичей закончена, делаем git reset на коммит до наших изменений
* коммитим все наши изменения в виде одного (или нескольких) коммита(ов)
* profit

При таком походе есть пара нюансов: пушить конечный коммит надо будет с форсом (--force), если кто-то вздумает смержить недоделанную ветку, то ничего хорошего это не принесёт.

Answer 5

[Vitsliputsli]

Делаете коммит с описание что сделали, что недоделали, начинаете коммит строкой "WIP: " (это стандартное сокращение означает "work in progress", многие системы игнорируют коммиты с таким началом описания).
После того как продолжите работу и сделаете логически завершенный коммит, то сделайте rebase и уберите этот коммит из истории (разумеется, если ветка это позволяет).

Answer 6

[Suor]

Синхронизируешь код с помощью дропбокса или чего-то аналогичного, коммитишь когда считаешь нужным.