@symfony_php

Страница 597 из 1418
Dinar
26.01.2018
18:40:22
Но у нас дженкинс. А какие минусы подхода?

Sergey
26.01.2018
18:40:54
Но у нас дженкинс. А какие минусы подхода?
дженкин это билд сервер. Как ты думаешь, почему это дело называется "постоянная интеграция"? что и с чем ты интегрируешь и что такое "постоянное"?

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
до этого - trunk based development пол годика
И, я так понимаю, было покруче?

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 котоыре что-то ломали... ну если и были - то там фиксилось в течении часа

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

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
Иногда надо инвестигейт делать как и когда сломалось.
практика показывает что линки на задачи редко в этом помогают.

Ну линкуются. Но смотреть 50 комитетов или один PR.
а как тебе поможет просмотр PR который был влит месяц назад?

и что ты этим инвестигейтом хотел узнать?

кто виноват тебе скажет 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 коммитов без код ревью (я так понимаю без) этому не сильно способствуют

Sergey
26.01.2018
18:53:58
Ну что за вопрос? :)Скажи, тебе никогда не приходилось гулять по изменениями, чтобы понять, когда это сломалось.
ммм... нет. Я обычно ищу в мастере или своей ветке конкретное место в коде, а дальше - git blame если нужно

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
Неверно понимаешь. У нас очень жесткое Код ревью
сколько времени у тебя в среднем займет ревью 1-ого коммита с изменением 10-ти файлов или 50-ти коммитов с изменениями 50-ти файлов? какой процент факапов ты не заметишь?

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

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 позволяют даже по части файла чекать изменения и строки и т.д. никаких проблем с инвестигейтом

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
Иногда проще вернуть как было, чем делать заново.
чем тебе тут помогут PR или джира? у тебя ж никто историю коммитов не отменяет. Более того - делать git rever маленьких коммитов проще

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

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

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

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
В данной статье я увидел только 1 против фича бренчей - конфликты. ВСЕ!
I'll go so far as to discount textual conflicts for the purposes of this article. The problem I worry more about is a semantic conflict.

Вот серьезно. Я просто не имею опыта в транке.
суть не просто в том что бы опыт был, нужно в целом уметь анализировать имеющиеся решения)

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

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

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