Skip to content
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

Pull Request updated. Finished review on Section '07-git-tools' #144

Merged
merged 3 commits into from
Sep 10, 2019

Conversation

DiegoFRamirez
Copy link
Contributor

@DiegoFRamirez DiegoFRamirez commented Sep 6, 2019

Hola @andres-mancera. Envío un Pull Request sólo para consulta. He encerrado en un comentario la última oración del párrafo final de la sección book/07-git-tools/sections/reset.asc. A mi parecer hay algo ahí que no debería estar. Parece sacada he insertada de otro contexto. Cuando puedas, por favor confírmamelo, pues si estoy en lo correcto, creo deberíamos evaluar eliminarla.
Gracias.
Un abrazo.

@DiegoFRamirez
Copy link
Contributor Author

DiegoFRamirez commented Sep 8, 2019

Hola de nuevo. Disculpas por continuar molestando. Ocurre que tengo un commit con la revisión del resto de la sección 07-git-tools. Sin embargo, me gustaría poder resolver la duda anterior, para ya acometerla también en el mismo Pull Request.
Me disculpo por las molestias nuevamente y aguardo vuestra respuesta.
Un abrazo.

Esta función calculará la ciudad y el país del visitante desde la dirección IP. Para activar esta función, debes descargar la base de datos GeoIP y configurar la clave de la API de Google. Por favor, vea "Más información"
**[:endig]>>**
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tienes razón. Esto no tiene sentido acá. Aquí está el contenido original de la versión es inglés:

https://git-scm.com/book/en/v2/Git-Tools-Reset-Demystified

¿Podrías por favor remover esto y asegurarte de que la traducción sea coherente?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok. Lo reviso y lo arreglo. Gracias. 👍

@DiegoFRamirez DiegoFRamirez changed the title Don't merge this Pull Request. It is a commit for consult Pull Request updated. Finished review on Section '07-git-tools' Sep 9, 2019
@DiegoFRamirez
Copy link
Contributor Author

Hola nuevamente @andres-mancera. Agregué el cambio sugerido en:

book/07-git-tools/sections/reset.asc

Unifiqué todo en un solo Pull y actualicé el título. Esta sección en particular es bastante verbosa, por lo cual quizás haya sido excesivo reunir el 50% en un solo commit. Espero no complique la revisión. Hice varios cambios en: estructura, términos utilizados, corrección de ambigüedades, nuevamente unifiqué el trato al lector por capítulo y modifique alguna que otra estructura sintáctica y/o gramatical. En cuanto al término commit he comenzado a encontrar referencias por género opuestas, por lo tanto, y tomando en cuenta que en los capítulos anteriores se le da un trato masculino, lo he corregido en aquellos casos donde se utilizaba el pronombre 'la' en lugar de 'el'.
Quedo a disposición en caso de que haya cometido algún error para corregirlo a la brevedad.
Saludos.

@andres-mancera andres-mancera merged commit 1b5bf22 into progit:master Sep 10, 2019
@andres-mancera
Copy link
Member

@DiegoFRamirez : Te agradezco nuevamente por tu PR. ¡Qué cantidad de trabajo la que estás haciendo en la revisión de la traducción de este libro!

@DiegoFRamirez
Copy link
Contributor Author

@andres-mancera: Nada que agradecer. Me alegro que pueda ayudar a que aumente la consistencia de la traducción. Sé que falta, porque aún no he llegado a la parte que me llevó a solicitar poder colaborar. Seguimos avanzando y veremos con lo que nos encontramos.
Saludos.

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.

3 participants