
Dinar
26.01.2018
18:40:22
Но у нас дженкинс. А какие минусы подхода?

Sergey
26.01.2018
18:40:54

Evgeniy
26.01.2018
18:41:18

Sergey
26.01.2018
18:41:30

Google

Sergey
26.01.2018
18:41:33
https://trunkbaseddevelopment.com/

Dinar
26.01.2018
18:41:40

Sergey
26.01.2018
18:42:10

Dinar
26.01.2018
18:42:26
Спасибо. Буду читать.
А у вас что?

Sergey
26.01.2018
18:42:45
сейчас - фичабрэнчи) не могу пока уговорить отдельных лиц отказалсять и перейти на CI
до этого - trunk based development пол годика

Evgeniy
26.01.2018
18:43:29
фичабрэнчи - ветка под каждую задачу чтоли?

Sergey
26.01.2018
18:43:50
А у вас что?
в команде из 30-ти человек (10 бэкэндщиков, 20 тех кто пользовался, QA, фронтейндеры и прочие дизайнеры) trunk based показал себя ооочень хорошо

Dinar
26.01.2018
18:43:57

Sergey
26.01.2018
18:44:27

Sergey
26.01.2018
18:44:30
у вас в мастере всегда рабочий код?

Sergey
26.01.2018
18:45:00

Google

Sergey
26.01.2018
18:45:01
ну скажем WIP который чет ломает, вы льете?
я про незаконченные фичи

Sergey
26.01.2018
18:45:45
небыло WIP котоыре что-то ломали... ну если и были - то там фиксилось в течении часа

Dinar
26.01.2018
18:45:55

Sergey
26.01.2018
18:46:02
я в ноябре на общих сборах топил за фича бранчи, проходит 2 месяца и я заявляю что буду топить за противоположность

Sergey
26.01.2018
18:46:03

Sergey
26.01.2018
18:46:09
СТО сделал вот так

Dinar
26.01.2018
18:46:31
нет, наоборот
Так релиз бранч. В него ж вмердживать только когда весь мастер рабочий.

Sergey
26.01.2018
18:46:45

Sergey
26.01.2018
18:46:55
в идеале ты по сути разрабатываешь в проде)
CI/CD

Dinar
26.01.2018
18:47:01
Ну так фичи не готовые же могут в нем быть.

Sergey
26.01.2018
18:47:10

Sergey
26.01.2018
18:47:15
фича флаги, абстрактные ветки

Dinar
26.01.2018
18:47:19
Есть такси которые не один день сидишь. А между ними баги фиксить.

Sergey
26.01.2018
18:47:54
ну то есть как, сломаться что-то может

Dinar
26.01.2018
18:48:28
А ещё не получится смотреть фича пулл реквесты
В джире

Sergey
26.01.2018
18:48:47

Google

Dinar
26.01.2018
18:48:51
Те изменения которые относились к фиче

Sergey
26.01.2018
18:49:12
ну и опять же - линкуй задачи к коммитам - типа "PROJ-123: Add option field for bla bla"

Dinar
26.01.2018
18:49:14
Иногда надо инвестигейт делать как и когда сломалось.
Ну линкуются. Но смотреть 50 комитетов или один PR.

Sergey
26.01.2018
18:49:47
и что ты этим инвестигейтом хотел узнать?
кто виноват тебе скажет git blame
мне кажется что ты придумал этот юзкейс и на самом деле тебе не это нужно)

Sergey
26.01.2018
18:51:35
эм
заходишь в жиру
смотришь на TROLOLO-50

Sergey
26.01.2018
18:51:49
там список коммитов которые были влиты для нее
и выясняешь
после PR идет еще вагон фиксов как правило
которые не были заребейзены

Sergey
26.01.2018
18:53:03
Иногда надо инвестигейт делать как и когда сломалось.
1. ты ищешь не PR а место где сломано. Дальше ты смотришь git blame для этого места и смотришь коммит, он уже ведет на людей и задачу.
2. если тебе это надо делать часто - то на лицо более глубокие проблемы, которые возможно можно решить или как минимум минимизировать. И да, PR на 50 коммитов без код ревью (я так понимаю без) этому не сильно способствуют

Dinar
26.01.2018
18:53:23

Sergey
26.01.2018
18:53:58

Google

Sergey
26.01.2018
18:54:04
код ревью не панацея
и мало че решает

Dinar
26.01.2018
18:54:26
иногда блейм не дает ответа, так как чувак например делал это 80 лет назад и не помнит

Sergey
26.01.2018
18:54:37
для этого есть линк на задачу с описанием

Sergey
26.01.2018
18:54:42

Dinar
26.01.2018
18:54:53
Иногда удаленные строки не показывают блейм )

Sergey
26.01.2018
18:54:56

Admin
ERROR: S client not available

Dinar
26.01.2018
18:55:18
Они удалены и как раз сломали что-то. :)
Мне кажется, ты пытаешься оправдать фичу :)
Я считаю, что нужны все возможности.

Sergey
26.01.2018
18:56:12
как часто (сколько раз в год) тебе это нужно?

Dinar
26.01.2018
18:56:21
А еще говоришь "Вот тут будет понеудобнее, тут по больше проблем, но в целом-то все это фигня. Главное фича классная" Ж)

Sergey
26.01.2018
18:56:50

Dinar
26.01.2018
18:57:04

Alexander
26.01.2018
18:57:28
Phpstorm и git history позволяют даже по части файла чекать изменения и строки и т.д. никаких проблем с инвестигейтом

Sergey
26.01.2018
18:57:32

Google

Dinar
26.01.2018
18:57:40

Sergey
26.01.2018
18:58:03

Dinar
26.01.2018
18:58:26

Sergey
26.01.2018
18:58:43

Dinar
26.01.2018
18:59:36
Ну ок. Я допускаю, что я просто думаю в категориях фичабренчей и не представляю транк-разработка как ведется.

Sergey
26.01.2018
18:59:37
а не не оно... не найду сейчас

Dinar
26.01.2018
19:00:07
Но я совершенно уверен, что фичабренчи - рабочий и вполне крепкий тип

Sergey
26.01.2018
19:00:12

Dinar
26.01.2018
19:00:15
Да это уже кинул же ты :)

Sergey
26.01.2018
19:00:20

Dinar
26.01.2018
19:00:39
А фаулер если бы слушал других - не стал бы фаулером :)
Другие были бы фаулерами :)

Sergey
26.01.2018
19:01:11
> Indeed I see this as the decisive reason why Feature Branching is a bad idea. Once a team is afraid to refactor to keep their code healthy they are on downward spiral with no pretty end.
(с) Мартин Фаулер, сказ о семантических конфликтах и рефакторинге

Dinar
26.01.2018
19:01:27
В данной статье я увидел только 1 против фича бренчей - конфликты. ВСЕ!
Вот серьезно. Я просто не имею опыта в транке.

Sergey
26.01.2018
19:02:18

Dinar
26.01.2018
19:03:13
Я просто представляю себе это так:
Я решил сделать огромный рефактор. Я беру транк, делаю рефактор. КТо-то за это время назаливал изменений. У меня конфликты. Я их чиню. И пушу в транк. В чем разница? Кроме того, что я не могу быстро начать фиксить какой нить хотфикс?

Sergey
26.01.2018
19:03:26
а мне просто скучно и я накидываю что бы ты задумался - не являются ли те элементы процесса которые у тебя сложились не решением проблем, а ее прикрытием?