
Sergey
22.01.2018
19:03:49
1.1 версия вышла, для нее завели ветку

Sergey
22.01.2018
19:03:51
а... ну да
в целом так и есть

Sergey
22.01.2018
19:03:59
и там теги клепают на каждый релиз

Google

Sergey
22.01.2018
19:04:07
а ты монорепозитории курил?

Sergey
22.01.2018
19:04:17
типа вместо кучи репосов один?

Sergey
22.01.2018
19:04:21
да
в случае symfony это выглядит логично
но я про ситуацию когда у тебя там реакты, ангуляры, пыхи, джавы, котлины, андроиды... и все вместе

Sergey
22.01.2018
19:04:52
та ну не

Sergey
22.01.2018
19:05:03
все крупные чуваки так делают)
https://cacm.acm.org/magazines/2016/7/204032-why-google-stores-billions-of-lines-of-code-in-a-single-repository/fulltext

Sergey
22.01.2018
19:05:36
надо покурить
не задумывался в принципе

Sergey
22.01.2018
19:08:03
ну короч я сча даунгрейднулся на фичабрэнчи (надеюсь временно)
а, да, единственный момент - код ревью
с trunk-based dev у тебя чуть осложняется этот вопрос... я у себя решал это попытками парное программирование двигать, ну и пост фактум ревьювя код

Google

Sergey
22.01.2018
19:09:08
но как быть в распределенной команде пока хз

Sergey
22.01.2018
19:09:11
ну по факту у нас есть фича бранчи
мы в них кодим, каждый тикет
потом реквест на мерж
мы ревьювим
сливаем в общую ветку и дальше это тестят там

Sergey
22.01.2018
19:09:40
ну то есть у тебя может быть ситуация когда разработчик пилит фичу 3-4 дня
и только потом вливает

Sergey
22.01.2018
19:09:47
он всегда ребейзит
от общей ветки
и ниибет

Sergey
22.01.2018
19:10:01
это да, но как бы... это всеравно не то

Sergey
22.01.2018
19:10:14
если он эти 3-4 дня не делал ребейс, то сам себе буратино)

Sergey
22.01.2018
19:10:34
ну а бывает такое что за день вообще никто не замерджил ничего в мастер?

Sergey
22.01.2018
19:11:02
на выходных)

Sergey
22.01.2018
19:11:16
ну короч я более радикальный - либо ты делаешь PR и его ревьювят и валивают в тот же день (ну то есть каждый день - новый PR)
либо без веток вообще

Sergey
22.01.2018
19:11:26
у нас на ревью мало времени выделяется
иногда это пост-ревью

Google

Sergey
22.01.2018
19:11:45
ну типа ты поставил в известность людей

Sergey
22.01.2018
19:11:46

Sergey
22.01.2018
19:11:49
они посмотрели код
и все

Sergey
22.01.2018
19:11:59
ибо мне порой приходится сутки ждать пока доберутся до PR)

Sergey
22.01.2018
19:12:11
ну такое только если ты в чужой код льешь
ну, где у тебя контекст не твой
скажем я как бекенд полезу на фронт и чет там поменяю
то буду ждать апрува

Sergey
22.01.2018
19:13:25
ай не, я лучше буду если лезу заранее договариваться о сессии парного программирования
так проще, быстрее, и в следующий раз я уже смогу сам это сделать

Alan
22.01.2018
19:20:08

Sergey
22.01.2018
19:20:39
вот главный аргумент против, это чтобы фичи быстрее в прод уходили
хочется хуяк-хуяк и в прод
не ожидая всего релиз цикла

Sergey
22.01.2018
19:27:34
я хочу добиться того что деплоймент != релиз
и мутить CD полноценный
хотьбы раз...
но для этого надо сначала CI полноценный и покрытие кода тестами увеличить (end-to-end для начала намашнить)

Bohdan
22.01.2018
20:35:29
хренюлю с фосюзером и переопределенным registrationcontroller никто не ловил недавно?

Google

Alex
22.01.2018
22:25:37
Ну я конечно тоже в чем-то приверженец анархизма, но про фичабренч ветке не согласен народ )))

Admin
ERROR: S client not available

Sergey
22.01.2018
22:25:57
вопрос
юзаете LoggerAwareTrait
или инжектите логгер в конструктор?

Alex
22.01.2018
22:26:16
Если это продукт не ОСС то мастер это и есть продакшн который можно каждый раз после изменения деплоить
а изменения через PR с форка т.е. таже фичабренч в форке, которая прошла rebase с текущим мастером и естественно уже грин по всем проверкам

Andrew
22.01.2018
22:29:46

Sergey
22.01.2018
22:40:37

Alex
22.01.2018
22:40:43

Sergey
22.01.2018
22:40:57
ну то есть.... я вот как человек который работал и так и эдак могу сказать что master и код фриз в release ветку работает эффективнее

Alex
22.01.2018
22:41:20
а зачем?
чтоб изалировать WIP от рабочего и проверенного кода

Sergey
22.01.2018
22:41:34

Sergey
22.01.2018
22:41:45
https://martinfowler.com/bliki/FeatureBranch.html

Sergey
22.01.2018
22:43:01
чтоб изалировать WIP от рабочего и проверенного кода
1. как долго у тебя в среднем живут ветки?
2. как часто появляются конфликты?
3. насколько у вас распространен рефакторинг (постоянное переименование классов, методов, реструктуризация)
4. насколько у вас с авто тестами?

Alex
22.01.2018
22:43:02

Sergey
22.01.2018
22:43:22
(я сча не про дженкинсы а про подход)

Google

Alex
22.01.2018
22:43:45
github.com/akeneo/pim-community-dev
https://ci.akeneo.com/blue

Sergey
22.01.2018
22:44:20

Alex
22.01.2018
22:44:44
Я понимаю что подход и не важно что начинает рулить таски проверки

Sergey
22.01.2018
22:44:45
это когда у тебя все мерджат в mainline хотя бы раз в день и все такое
ад

Alex
22.01.2018
22:47:06
нет, там какие то тестовые poc ветки которые out of focus вообще, все движение через PR

Sergey
22.01.2018
22:47:10
WIP отделен от "проверенного" кода, но при этом нет необходимости в изоляции каждой фичи