Закрыть первый PR, поскольку в производной ветви тоже произошли изменения

#git #github

#git #github

Вопрос:

Допустим, я поработал над task1 веткой и после этого нажал на нее, а затем создал запрос на извлечение. Никаких конфликтов слияния с master .

Поскольку вышеупомянутый PR не был объединен с master , я создал 2-ю ветвь как task2 использующую task1 ветвь следующим образом:

 git checkout -b task2 task1
  

После этого я нажал на эту ветку следующим образом:

 git push --set-upstream origin task2
  

В этой ветке также нет конфликтов слияния. Итак, мой вопрос в том, нужна ли мне task1 ветка PR сейчас? Поскольку я получил task2 ветвь от task1 надеюсь, я смогу close task1 PR ‘s’ нет? Придется ли мне столкнуться с какими-либо проблемами?

Комментарии:

1. Это зависит от того, какова ваша цель. Если вы закроете исходный PR и создадите новый PR из task2 , он будет содержать объединение всех изменений. Это то, чего вы хотите?

2. Да, именно это мне и нужно. Мне нужна вся работа ветвей task1 и task2 и объединить ее в master в конце. Будет ли этот рабочий процесс иметь какие-либо побочные эффекты? @OliverCharlesworth

3. Если другие разработчики уже просмотрели ваш исходный PR (и добавили комментарии и т.д.), То я полагаю, они могут быть раздражены тем, что им приходится делать это снова! И в некоторых командах не рекомендуется использовать большие многофункциональные PR. Но я думаю, что в конечном счете это «культурные» проблемы, а не технические.

4. Большое спасибо за отличную информацию. @OliverCharlesworth

Ответ №1:

Закройте исходный PR и создайте новый PR из задачи 2, он будет содержать комбинацию всех изменений. Никаких других эффектов, кроме этого, для вашего рабочего процесса не будет.