Детали вашего рабочего процесса ветвления / слияния должны решаться вашей командой, так что действительно вопрос, который задают, основан на мнении и, вероятно, должен быть закрыт.
В духе того, что он немного более полезен, чемчто: ближайший потенциально-фактический вопрос был бы, каковы плюсы и минусы такого слияния.
Как вы заметили, вы получите больше работы по слиянию с путичеловеком, знакомым с намерением изменений в ветви функций;это PRO
Но с другой стороны, этот человек может не быть наиболее знакомым с намерением других изменений в develop
;это CON, который, вероятно, отменяется с помощью вышеуказанного PRO
. Вы получите дополнительное слияние в истории.Некоторые люди патологически против этих (и, в большинстве случаев, большинства) коммитов слияния.Это граничит с «мнением», поэтому я просто скажу, что это разница.
Вы проверяете объединенное состояние на ветви;это ПРО; но это должно быть умеренным, потому что вы все еще не знаете, что больше вещей не будут объединены, чтобы развиваться впереди вас.(Если открыто несколько PR, это наверняка произойдет со всеми, кроме одного.)
Альтернативой, со своими плюсами и минусами, будетперебазируйте вашу ветку функций в совет по разработке перед пиаром.Функционально это похоже на вышеприведенное, но плюсы и минусы немного отличаются.
В результате потенциально много состояний фиксации, которые не были протестированы.Если у вас обычно есть хороший автоматизированный набор тестов, то создание непроверенных состояний, которые могут не пройти тесты, является потерей возможности устранения неполадок.
Он не только избегает фиксации «обратного слияния», но может (для PR, которые не складываются) также избежать форвард-слияния.Так что, если вам действительно не нравятся коммиты слияния, это профессионал.
Еще одна альтернатива - выполнить слияние, чтобы просто выяснить какие-либо конфликты и протестировать объединенное состояние, а затем вернуться назад.слияние из функциональной ветви.Это позволяет избежать «лишнего» коммита слияния, но это означает (если вы не можете использовать git rerere для смягчения этой проблемы), что вам придется разрешать любые конфликты дважды.
Итог
Существует множество различных рабочих процессов и множество разных мнений о том, что является «правильным».Только ваша команда может решить, что работает для вашей команды, получив информацию о вариантах и их последствиях.