Skip to content

mi aporte txt de prueba#111

Merged
aalbagarcia merged 2 commits into
aprendegit:masterfrom
wibastidas:commitWilliam
Dec 4, 2017
Merged

mi aporte txt de prueba#111
aalbagarcia merged 2 commits into
aprendegit:masterfrom
wibastidas:commitWilliam

Conversation

@wibastidas
Copy link
Copy Markdown

No description provided.

@aalbagarcia
Copy link
Copy Markdown
Member

Hola @wibastidas

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

@wibastidas
Copy link
Copy Markdown
Author

Hola @aalbagarcia gracias por responder. La verdad me quedan algunas dudas sobre este segundo commit.

La razón del commit es porque debo sincronizar la rama que cree en mi fork con la rama original para que puedas aceptar el pull request?

El commit que me pides debo hacerlo a la rama que creé dentro de mi fork correcto?

No es necesario volver a crear un pull request?

@aalbagarcia
Copy link
Copy Markdown
Member

Hola @wibastidas

La razón del commit es porque debo sincronizar la rama que cree en mi fork con la rama original para que puedas aceptar el pull request?

El motivo por el que os pido que me enviéis un nuevo commit es para que veáis como github detecta los cambios en la rama de tu fork (desde la que iniciaste el pull-request) cuando hace push y los muestra aquí.

El commit que me pides debo hacerlo a la rama que creé dentro de mi fork correcto?

¡¡Correcto!!

No es necesario volver a crear un pull request?

¡¡No!! basta con que hagas push, como ya lo has hecho.

@aalbagarcia aalbagarcia merged commit 91186bc into aprendegit:master Dec 4, 2017
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