@symfony_php

Страница 589 из 1418
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)

либо без веток вообще

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
Если это продукт не ОСС то мастер это и есть продакшн который можно каждый раз после изменения деплоить
Зависит от флоу, например в gitlab flow для продакшна рекомендуется отдельная ветка production

Alex
22.01.2018
22:40:43
Зависит от флоу, например в gitlab flow для продакшна рекомендуется отдельная ветка production
верно, они от gitflow это кажись зацепили... условно продакшн ветка может называться как угодно, но разделять работу девов все равно нужно по ветка, пускай это будет не фича а микро таск... иначе потекут мозги от головной боли где что кто поменял )))

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

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

Sergey
22.01.2018
22:41:34
чтоб изалировать WIP от рабочего и проверенного кода
то есть continious integration это не православно?

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
то есть continious integration это не православно?
CI то да, но даже когда каждая ветка таска проходит CI иногда после слияния могут возникать лажи

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
https://ci.akeneo.com/blue
так, CI это процесс а не тулы

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

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

ад

чтоб изалировать WIP от рабочего и проверенного кода
повторюсь, если у тебя WIP живет в мастере а проверенный код - это результат код фриза (релиз брэнч), то чем это плохо?

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

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

Страница 589 из 1418