SG
особенно если речь идет про какой-то рефакторинг, как уже упоминали
Size: a a a
2021 April 21
AR
pr stacking, какой-то последовательный план, etc
AR
понятно что иногда не избежать изменения в 30 файлах, но это могут быть примитивные изменения
SG
могут, но ты потратишь кучу времени открывая на них отдельные МРы) а можно было бы просто сделать несколько коммитов с нормальным описанием
SG
да и контекст переключать не нужно, если хочешь в редакторе быстро посмотреть историю изменений, а иначе придется идти в браузер и в репозитории МР искать
DD
О, давно не было срачей про сквоши и ребейсы
VS
Я просто не розумію який профіт у видаленні історії. Навіть якщо вона не ідеальна.
AR
точно так же не понимаю зачем за историей следить
VS
Імхо, вже краще працювати без пр і комітити все в мастер просто. Це не жарт, люди так працюють і я вважаю що це норм підхід.
AL
мы не сквошим почти. У нас PR в основном это 1 коммит, иногда 2-3. И если это какие-то смысловые коммиты, то я просто делаю ребейс, и в основной ветке они просто идут один за другим.
AR
ну если команда достаточно дисциплинирована чтобы не плодить дурацкие “make eslint happy” коммиты в своих ветках, то можно и ребейзить
VS
Це краще ніж еслінт конфіг в коміті з якоюсь фічею
AR
хто ж так делает то
VS
Ну так тоді не розумію твій пойнт. Якщо це два різних коміти мають бути, то як сквош гарантує нормальні назви комітів.
AL
а, это я уже сам на код ревью смотрю, и если такие коммиты есть, то делаю squash & rebase в GH
Если же мне кажется, что в PR все-таки стоит выделить 2 и более отдельных коммитов, то я:
- стягиваю ветку себе
- кромсаю чейнжсеты как мне надо
- форс-пуш в ветку PR
- в GH делаю rebase
Если же мне кажется, что в PR все-таки стоит выделить 2 и более отдельных коммитов, то я:
- стягиваю ветку себе
- кромсаю чейнжсеты как мне надо
- форс-пуш в ветку PR
- в GH делаю rebase
AR
хочешь поменять эслинт конфиг — открой пулл реквест для этого, опиши что и зачем и обсуди с командой
AR
меня например бесит когда делают какую-то фичу/апи в пулл реквесте с баг фиксом
EO
у меня начинает складываться ощущение, что тут собрались выборки "большие проекты\команды" и "маленькие проекты\команды"
и каждый пытается доказать свой поинт
и каждый пытается доказать свой поинт
VS
Ну тут я тільки за
AL
+1 сдесь