Skip to content

Working on reference/index#17

Merged
raulcd merged 1 commit intopython:3.7from
carlos-bernad:reference/index
Mar 24, 2020
Merged

Working on reference/index#17
raulcd merged 1 commit intopython:3.7from
carlos-bernad:reference/index

Conversation

@carlos-bernad
Copy link
Copy Markdown
Contributor

No description provided.

@raulcd raulcd added the changes-requested Further changes have been requested to the PR label Mar 23, 2020
@carlos-bernad
Copy link
Copy Markdown
Contributor Author

Gracias Raúl, cambié el término antes de hacer el commit pero no revisé el artículo de delante :(.

Soy nuevo en las revisiones de cambios. Entonces, para corregir la petición de cambios he hecho un nuevo commit pero me habría gustado hacer otra cosa para no hacer ese commit y meter la modificación en el último commit que hice.
¿Es esto posible o como ya he hecho el PR, hacer un rebase no estaría permitido?
En resumen,
¿podría haber hecho lo siguiente?
git rebase -i HEAD~1
hacer el cambio
git add reference/index.po
git commit (+ comentario añadido)
git push origin HEAD

Gracias!

@raulcd
Copy link
Copy Markdown
Collaborator

raulcd commented Mar 23, 2020

Hola Carlos,
Eso es posible. Si quieres puedes hacerlo todavia pero tendrás que hacer force push a tu rama.
Si haces:
git rebase -i HEAD~2
Eso te permitirá modificar tus commits y te permitirá hacer squash (marca el segundo commit como squash). Eso te unirá los dos commits en uno. Después podrás hacer git push --force.
He encontrado éste post que explica lo que te comento:
https://blog.ona.io/general/2016/02/02/squashing-with-git-interactive-rebase.html

Update index.po

Minor fix due to gender issue
@carlos-bernad
Copy link
Copy Markdown
Contributor Author

Ya está, muchas gracias. Así voy descubriendo más partes del flujo de git. Falta que Travis dé el visto bueno, aunque en la página aparece como terminado correctamente.

@raulcd raulcd merged commit 9df4d9b into python:3.7 Mar 24, 2020
@carlos-bernad carlos-bernad deleted the reference/index branch March 25, 2020 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changes-requested Further changes have been requested to the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants