EB
В такой, в какой два разработчика договорились
Size: a a a
2021 April 06
ЕГ
Когда обе части фичи работают и их смержили в общую ветку
EB
Прямо буквально вот сегодня 2 разработчика договорились кто куда какую ветку будет лить код
O
так я вот сейчас хочу договориться как правильно
O
я пока не понял, это будет отдельная ветка или дев или что
EB
А зачем тебе понимать? :) ты хочешь научить разработчиков как им работать?
ЕГ
Сделайте отдельную ветку под фичу, самый простой вариант. Бэкер и фронтер будут делать в своих, а потом туда смёржат рабочий вариант
O
Ок. Скорее всего мержей будет несколько
ЕГ
Пусть будет. Чего вы так боитесь мержей?) Просто часто делаете на них акцент, будто это плохо
O
Вот это все про невозможность изменения контракта и что с первого раза все пишут хорошо, мне кажется имеет отношение к идеальному миру.
O
Я хочу понять когда происходит ревью МР
O
Это основной вопрос
O
Сейчас получается, что есть ветка на story, Есть две ветки на сабтаски. Они обе мержатся в ветку story
O
Видимо в этот момент делается ревью
O
потом приходят правки, делаются в ветках сабтасок, опять мержатся в ветку со стори - делается новое ревью, так?
ЕГ
Можно так, можно когда сторя мержится в стэйдж\релизХ\дев ветку. Как разработчики договорятся и как им будет удобнее, на самом деле
O
Если делать когда сторя мержится в дев, то в ней будет код от обеих сабтасок
ЕГ
Например, если вмержились в сторю, а потом сторю в дев льют, делается ревью и прилетают предложения правок, то разработчику ничего не мешает зайти в эту ветку и прям там поправить код и пушнуть в этот же МР
O
А это может быть слишком много для одного ревью. Собственно имеено это хочется разделить
ЕГ
Ну тогда напрашивается ревью при мерже сабтаски в сторю)