AL
Ага, имхо в большой команде добиться чистых коммитов очень сложно. Всегда найдется какой-то умник, который херню в истории устроит.
Size: a a a
2021 April 21
AL
И если в маленькой команде кто-то заметит и подметет за ним, то в большой изменения идут так быстро, что сказать всем “стоп, у нас лажа, никому не пушить в мастер, пока мы не разберемся” нельзя
EO
да, этого сложно добиться
поэтому на малых проектах имеет смысл черипик, ребейс, вытирание пыли с истории (я сам так делаю на своих же проектах)
а на больших проектах есть куча веток от интерна до всяких security assesments команд, которые болт клали на историю (сквош единственный метод держать это организованно)
поэтому на малых проектах имеет смысл черипик, ребейс, вытирание пыли с истории (я сам так делаю на своих же проектах)
а на больших проектах есть куча веток от интерна до всяких security assesments команд, которые болт клали на историю (сквош единственный метод держать это организованно)
SS
Якщо цей баг заважає фічі?
Новий пр. Мердж. І тоді продовжувати?
Новий пр. Мердж. І тоді продовжувати?
AL
Тебе никто не мешает делать бранч от бранча и так же его отдельным Пиаром закидывать
AR
^ this
AR
всегда можно себя разблокировать, если какие-то пулл реквесты висят чуть дольше чем хотелось бы
SS
Навіщо? Типу щоб про цей багфікс знали? В чому профіт?
AL
дв том, что когда я git blame смотрю, я вижу зачем ты трогал вот ту строку кода.
AL
А так имеем картину: поменяна логика выгрузки отчетов, причем как-то неочевидно. А коммит мессадж говорит “сделал АПИ для регистрации баянов”
И ты такой: Чтоооо?!
И чувак тоже не помнит уже.
И ты такой: Чтоооо?!
И чувак тоже не помнит уже.
RV
Так же помогает не раздувать размер PR. Ведь его нужно заревьювить а потом протестировать перед тем как замерджить. Легче заревьювить и протестировать отдельно багфикс, а потом отдельно фичу.
SS
Можна робити тоді ребейс інтерактивний і лишити 2 коміта для злиття фічі далі.
AL
вот ты локально у себя и делай их перед тем, как PRы открывать
AL
багфиксы - отдельно
фичи - отдельно
рефакторинги / техдолг / гигиена - отдельно
Отдельные коммиты и отдельные PRы
фичи - отдельно
рефакторинги / техдолг / гигиена - отдельно
Отдельные коммиты и отдельные PRы
AR
PR, помимо прочего, это кусочки комуникации с командой
VS
а як потім дізнатися який пр за яким мерджити?
AL
А ты их не одновременно открывай. Ветки пуш, а PR не заводи
VS
а, ок, норм
RV
Мы часто открываем их с разными тагретами.
MR1: багфикс -> мастер
MR2: фича -> багфикс
Помогает видеть релевантный дифф в MRах. А когда MR1 (багфикс) замерджен, то переставляем таргет у MR2 (фича) на мастер. Ну и регулируется очередностью тикетов в джире
MR1: багфикс -> мастер
MR2: фича -> багфикс
Помогает видеть релевантный дифф в MRах. А когда MR1 (багфикс) замерджен, то переставляем таргет у MR2 (фича) на мастер. Ну и регулируется очередностью тикетов в джире
VS
кул, дякую