SB
jooq ?
Size: a a a
2019 May 16
ch
оно спасибо
SK
Я конечно не спец в git, но думаю все таки надо делать частый локальные каммиты, чтобы можно было в любой момент откатить код, а на сервер загружать только финальный каммит.
Частые коммиты удобны при разработке фичи с нуля на относительно чистом проекте. Один коммит на всё неизбежен при рефакторинге легасятины, попытка его разбить чаще всего приводит к геморрою. Как при мерже, так и при сквоше в апстриме фича так и так одним коммитом ляжет. А до апстрима ИМХО никого не должно волновать как девелоперу удобней менеджить и трэкать его работу на его же личной, девелоперской ветке.
AT
Частые коммиты удобны при разработке фичи с нуля на относительно чистом проекте. Один коммит на всё неизбежен при рефакторинге легасятины, попытка его разбить чаще всего приводит к геморрою. Как при мерже, так и при сквоше в апстриме фича так и так одним коммитом ляжет. А до апстрима ИМХО никого не должно волновать как девелоперу удобней менеджить и трэкать его работу на его же личной, девелоперской ветке.
Тут важно как это все ревьювить.
Код без ревью в мастере- 100% геморрой.
Код без ревью в мастере- 100% геморрой.
SK
Тут важно как это все ревьювить.
Код без ревью в мастере- 100% геморрой.
Код без ревью в мастере- 100% геморрой.
Ну это само собой. Как ревью коррелирует с количеством коммитов?
AT
Ну как- каждый коммит должен содержать что-то.
Тут рефакторинг, тут новый интерфейс, тот правка старого.
Тут рефакторинг, тут новый интерфейс, тот правка старого.
AT
А когда коммиты логически сгруппированы- то зачем их сквошить? Сквошат когда там поток мыслей- сделал, поправил багу в предыдущем, поправил багу в правке баги 🙂
SK
Ну как- каждый коммит должен содержать что-то.
Тут рефакторинг, тут новый интерфейс, тот правка старого.
Тут рефакторинг, тут новый интерфейс, тот правка старого.
Кому должен? Если речь об информативности, то имхо лучше требовать описание RCA в дескрипшене, и не выносить моск девелоперу тем как ему следует разбивать его работу. Дескрипшн pr-а в отличии от коммитовой истории тоже можно ревьювить и править ремарками.
AT
У нас при переезде с Gerrit на gitlab все реквесты пропали. А кота есть.
AT
Коммиты
SK
У нас при переезде с Gerrit на gitlab все реквесты пропали. А кота есть.
А обязательно было геррит грохать?
AT
А обязательно было геррит грохать?
Конечно. Зачем г..но мамонта хранить?
SK
Конечно. Зачем г..но мамонта хранить?
Ну вот какое отношение такой и результат)
AT
Так коммиты есть, ничего не потеряно. Зато работать стало удобнее
SK
Так коммиты есть, ничего не потеряно. Зато работать стало удобнее
Не верю что удобнее. Моя локальная бранча - моё личное пространство. Попытки лезть в моё личное пространство ведёт к чему угодно, кроме удобству.
SK
И говоря о ревью - представьте что вам пришла фича, а к комитам есть претензии. Как править ремарки? Историю переделывать?
AT
Не верю что удобнее. Моя локальная бранча - моё личное пространство. Попытки лезть в моё личное пространство ведёт к чему угодно, кроме удобству.
Пока личная - да. Как на ревью - будь добр, создай коммиты заново. Да, это сложно. Зато потом это можно понять
SK
Если это локальный сквош - я в своём праве. Пока я на локальной ветке, никто не имеет права вмешиваться. Я иак считаю. Если придёт вам на ревью непонятная хрень - ваше право послать меня. А пока я сижу локально - все вон.
AT
И говоря о ревью - представьте что вам пришла фича, а к комитам есть претензии. Как править ремарки? Историю переделывать?
Иногда да. Иногда правки идут как рефакторинг. Однозначного решения нет
SK
Я б к вам не пошел.