Как изменить базовую ветвь закрытого / объединенного пул-запроса?

Question

Я разветвил репозиторий, внес некоторые изменения, зафиксировал их и отправил запрос на извлечение. PR был принят, и изменения были объединены в репозиторий верхнего уровня. Однако позже я понял, что допустил ошибку при выдаче PR из основной ветки моего форка, вместо создания отдельной ветки для внесения всех изменений и отправки оттуда PR.

Мне было интересно, есть ли способ изменить базовую ветвь закрытого / объединенного PR?

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

Буду благодарен за любую помощь, которую вы можете оказать.

Редактировать 1: я пометил ответ @ bk2204 как принятый ответ, поскольку автор далее рассмотрел возможные последствия удаления вилки. Хотя, ответ @msanford также был чрезвычайно полезен, и я хотел бы отметить оба ответа. Ура!

Answer 1

Прежде всего, звучит так, как будто вы говорите о желании изменить ветвь, связанную с запросом на получение, вместо базовой ветки (ветвь, в которую он будет объединен).В любом случае, невозможно изменить ветвь объединенного запроса на извлечение;запрос на получение считается выполненным, и изменить его невозможно.

В этом случае нет необходимости удалять вилку из-за использования неправильной ветви.Вы можете изменить ветку master, как пожелаете, теперь, когда запрос на закрытие закрыт;история архивируется в главном репозитории и на стороне GitHub, и дальнейшие изменения в вашей ветке master не будут влиять на запрос извлечения.

Однако, если вы хотите удалить ветку masterили ваш ответвление по другим причинам, вы можете сделать это без негативных последствий для вышестоящего проекта или запроса на получение.

Answer 2

Хорошо, что вы заметили и хотите что-то с этим сделать.

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

Чтобы вникнуть в то, о чем вы беспокоитесь, ветки - это просто полезная абстракция для объединения серии коммитов. Тот факт, что вы отправили пиар из собственного master вместо ветки патча, который вы разветвляли master, не имеет значения. Предполагая, , что вы намеревались основать свою ветвь патча из master, ветки патча, и ваш мастер в конечном итоге будет одним и тем же, поскольку они содержат идентичную историю .

Если бы ваш патч входил в ветку функций в репозитории верхнего уровня , которая не содержала работы с master (по какой-либо причине), дополнительные изменения также проявились бы в вашем PR и и вы, и вышестоящий обозреватель заметили бы.

Для полноты: если вам действительно нужно исправить работу, которая была включена по ошибке, вероятно, потребуется новая последовательность изменений в новом запросе на извлечение.