Поправка к пиару и закрытие оригинала - PullRequest
Новая
       64

Поправка к пиару и закрытие оригинала

0 голосов
/ 09 декабря 2018

Я проверил местное отделение GitHub PR согласно этому описанию .Он предложил 

git fetch origin pull/ID/head:BRANCHNAME

В этой локальной ветке я применил некоторые исправления и хочу

  • объединить все коммиты с мастером (как исходный PR, так и мой локальный)
  • закройте исходный PR, чтобы было видно, что о нем позаботились (и оценили)

Какие соответствующие шаги необходимо предпринять?

Я рассматриваюперенесите локальную ветвь в новую удаленную ветвь, создайте новый PR и объедините его с сообщением «закрывается # original PR».Будет ли это работать?Какие неблагоприятные последствия это будет иметь?Могу ли я закрыть исходный PR с помощью сообщения слияния, или это должен быть коммит?

1 Ответ

0 голосов
/ 09 декабря 2018

Я бы сказал, что если ваши изменения в этом PR больше похожи на косметические \ небольшие исправления, чем вы должны:

  • Извлечь исходную ветку PR (клонируйте fork , чтоСоздан автор PR) git clone https://github.com/USERNAME/REPO && cd REPO && git checkout BRANCHNAME
  • Внесите свои изменения
  • Нажмите его непосредственно в репо форка
  • Изменения появятся в оригинальном PR
  • Объедините оригинальный PR

Так что, вероятно, вам не нужно создавать новый PR для этого, просто измените существующий вместо него.

См. Страницу Github для справки https://help.github.com/articles/committing-changes-to-a-pull-request-branch-created-from-a-fork/. Хотя, чтобы иметь возможностьдля фиксации в ветке форка необходимо, чтобы создатель PR установил флажок «Разрешить редактирование от сопровождающих».

Но в других случаях (например, если оригинальный PR был каким-то грязным или ожидал долгое время и требует усилий для его перебазирования)сейчас, или автор не поставил «Разрешить редактирование от сопровождающих»), предложенный вами подход также отлично подходит, нет ничего плохого в создании нового PR и объединении его вместо исходного.В этом случае я бы порекомендовал вам просто прокомментировать оригинальный PR, такой как «Перенесен на новый PR # 123, закрывая этот», чтобы было понятно, что вы делаете.

...