Skip to content

Aprendiendo a hacer pull requests#118

Merged
aalbagarcia merged 2 commits into
aprendegit:masterfrom
Ana-Grau:master
Jan 3, 2018
Merged

Aprendiendo a hacer pull requests#118
aalbagarcia merged 2 commits into
aprendegit:masterfrom
Ana-Grau:master

Conversation

@Ana-Grau
Copy link
Copy Markdown
Contributor

@Ana-Grau Ana-Grau commented Jan 2, 2018

Estoy haciendo un curso sobre forking y quiero aprender a hacer pull requests. Me aceptarian los cambios en el proyecto?

@aalbagarcia
Copy link
Copy Markdown
Member

Hola @Ana-Grau

Para que te acepte el pull request, debes enviarme un commit después de este comentario ¡quedo a la espera! :-)

@Ana-Grau
Copy link
Copy Markdown
Contributor Author

Ana-Grau commented Jan 3, 2018

Crea que el commit venia en el pull request. Es el 89fd9b8

89fd9b8

@aalbagarcia
Copy link
Copy Markdown
Member

@Ana-Grau Sí, es correcto. Cuando hiciste el pull request me enviaste un commit.

Lo que quiero es que me envíes otro commit. El ejercicio pretende que veas no sólo cómo hacer un pull request (cosa que ya has hecho) sino además ver cómo puedes seguir trabajando en el mismo pull request haciendo modificaciones sobre lo que ya has subido.

Por ejemplo, en el fichero Prueba.txt, puedes añadir otra línea más. Haces un commit y luego un push en tu rama master y verás que esas nuevas modificaciones aparecen aquí.

@Ana-Grau
Copy link
Copy Markdown
Contributor Author

Ana-Grau commented Jan 3, 2018

Gracias por la aclaracion. Hice un cambio en otro archivo y lo mande. Aunque puedo rehacer commits tambien. O eso creo :)

@aalbagarcia aalbagarcia merged commit f1599d7 into aprendegit:master Jan 3, 2018
@aalbagarcia
Copy link
Copy Markdown
Member

aalbagarcia commented Jan 3, 2018

@Ana-Grau ¡enhorabuena! ya has hecho tu primer pull request. Espero que el ejercicio te haya sido útil ¡nos vemos!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants