Skip to content

Working on reference/index #17

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 24, 2020
Merged

Conversation

carlos-bernad
Copy link
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
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
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
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