¿Cómo manejar tu primera Pull Request en GitHub?

Soy relativamente nuevo en Git. Mi conocimiento hasta ahora incluye git add push commit. Eso es todo, y hasta ahora me ha servido bastante bien. :)

Sin embargo, esta mañana alguien me envió una solicitud de extracción, que como he leído significa que alguien bifurcó mi repositorio, hizo algunos ajustes y me está señalando diciendo: "Hola Sergio, hice algunas mejoras. Considerar para la rama master."

¿Estoy en lo cierto?

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

Si es así, ¿cómo puedo ¿fusionar fácilmente estos cambios de usuarios en la rama maestra del repositorio? ¿Qué comandos necesito para disparar? Más importante aún, ¿puede alguien explicar el proceso en términos sencillos a lo que tiene que suceder?

Gracias.

Author: Only Bolivian Here, 2011-04-20
Source

3 answers

Tienes la idea correcta. si ese conjunto de cambios te parece útil, entonces probablemente quieras fusionarlo en tu repositorio.

Github ofrece una buena guía para responder a las solicitudes de extracción: https://help.github.com/articles/using-pull-requests Empezaría con eso y ver si tiene alguna pregunta más específica sobre el proceso.

 14
Author: Jonah,
Warning: date(): Invalid date.timezone value 'Europe/Kyiv', we selected the timezone 'UTC' for now. in /var/www/agent_stack/data/www/ajaxhispano.com/template/agent.layouts/content.php on line 61
2013-02-19 01:18:02

Github proporciona una pantalla de Cola de Bifurcación que te permite introducir los cambios en tu repositorio directamente en Github. Esta pantalla presenta cada confirmación bifurcada en verde o rosa dependiendo de si se aplicará limpiamente, y le permite especificar una nueva rama en su repositorio para fusionar los cambios. La URL de la cola de bifurcación es http://github.com// / forkqueue

Actualizar : Siga los siguientes pasos para fusionar la solicitud de extracción en su maestro repositorio:

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

Adaptado de la documentación de Github aquí: http://help.github.com/pull-requests/#merging_a_pull_request

Alternativamente, puedes usar la nueva "Solicitud de extracción de fusión", una nueva característica agregada a Github hoy (!): https://github.com/blog/843-the-merge-button

 6
Author: Dan Solovay,
Warning: date(): Invalid date.timezone value 'Europe/Kyiv', we selected the timezone 'UTC' for now. in /var/www/agent_stack/data/www/ajaxhispano.com/template/agent.layouts/content.php on line 61
2011-04-26 02:48:46

Una cosa que no se menciona explícitamente en la guía de solicitud de extracción de GitHub es cómo se debe aplicar dicha solicitud de extracción a su código:
Me gustaría que esos cambios se fusionaran:

  • de una manera rápida (que es simplemente mover el HEAD de mi rama a la siguiente parte n commits del parche)
  • o al menos sin ningún conflicto (el parche solo modifica/elimina líneas que no he tocado localmente o agrega nuevas líneas)

Si hay el más mínimo conflicto, es lo mejor es rechazar el parche, pedir al remitente que extraiga su propio código de nuevo, resolver cualquier conflicto localmente en su repositorio y hacer una nueva solicitud de extracción.
De esa manera:

  • ellos hacen el trabajo ("ellos "son los que te envían" pull requests", es decir, parches)
  • usted se beneficia;)
 3
Author: VonC,
Warning: date(): Invalid date.timezone value 'Europe/Kyiv', we selected the timezone 'UTC' for now. in /var/www/agent_stack/data/www/ajaxhispano.com/template/agent.layouts/content.php on line 61
2011-04-20 04:12:54