Как обработать ваш первый запрос на Pull на GitHub? - PullRequest
Новая
       5

Как обработать ваш первый запрос на Pull на GitHub?

26 голосов
/ 20 апреля 2011

Я относительно новичок в Git.Мои знания до сих пор включают git add push commit.Вот и все, и до сих пор он служил мне достаточно хорошо.:)

Однако сегодня утром кто-то прислал мне запрос на извлечение, который, как я прочитал, означает, что кто-то разветвил мой репозиторий, сделал некоторые изменения и пометил мне, говоря: «Эй, Серхио, я сделал некоторые улучшения. Рассмотрим ихдля основной ветки. "

Я прав?

https://github.com/sergiotapia/CherryTomato/pulls

Если так, как я могу легко объединить изменения этих пользователей в основную ветку хранилища?Какие команды мне нужно запустить?Что еще более важно, кто-то может объяснить процесс с точки зрения непрофессионала тому, что должно произойти?

Спасибо.

Ответы [ 3 ]

14 голосов
/ 20 апреля 2011

У вас правильная идея.если этот набор изменений выглядит полезным для вас, вы, вероятно, захотите включить его в свой репозиторий.

Github предлагает хорошее руководство по реагированию на запросы извлечения: https://help.github.com/articles/using-pull-requests Я бы начал с этого ипосмотрите, есть ли у вас более конкретные вопросы о процессе.

6 голосов
/ 20 апреля 2011

Github предоставляет экран Fork Queue, который позволяет вам вносить изменения в репо прямо на Github. На этом экране каждый разветвленный коммит отображается зеленым или розовым в зависимости от того, будет ли он применяться корректно, и позволяет указать новую ветку в репо для объединения изменений. URL для очереди ветвления: http://github.com///forkqueue

Обновление : Используйте следующие шаги для объединения запроса извлечения в ваш главный репозиторий: 

$ git checkout master
$ git remote add nakor git://github.com/nakor/CherryTomato.git
$ git fetch nakor
$ git merge nakor
$ git push origin master

Адаптировано из документации Github здесь: http://help.github.com/pull-requests/#merging_a_pull_request

В качестве альтернативы, вы можете использовать новый «Запрос на объединение», новая функция, добавленная в Github сегодня (!): https://github.com/blog/843-the-merge-button

3 голосов
/ 20 апреля 2011

Одна вещь, прямо не упомянутая в руководстве GitHub о запросах на получение , - это то, как указанный запрос на выборку должен применяться к вашему коду:
Я бы хотел, чтобы эти изменения были объединены:

  • в ускоренном режиме (это просто перемещение HEAD моей ветви к следующему n коммиту части патча)
  • или, по крайней мере, без каких-либо конфликтов (патч только изменяет / удаляет строкиЯ не трогал локально и не добавлял новые строки)

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

  • они выполняют работу («они» - это те, кто посылает вам «запросы на извлечение», то есть патчи)
  • вы прибыли;)
...