Clarify why two or one spaces and code formatting #1010
Conversation
| @@ -207,11 +207,37 @@ Luego nos vamos a nuestra rama local, confirmamos e impactamos esos cambios:: | |||
|
|
|||
| git checkout 3.8 | |||
| git merge upstream/3.8 | |||
| git push origin 3.8 | |||
| git push origin 3.8 | |||
There was a problem hiding this comment.
Sé que este cambio no es de acá...
pero ahora que lo leo...
Creo que con esto recomendamos actualizar las ramas locales, pero no la copia local del repositorio, quizás podríamos explicar como actualizar el fork y como actualizar la rama por separado?
Quizás entendí mal, pero para mi rama local sería "traduccion-xxxxxxxx" y no 3.8 del fork.
Con este mecanismo cuando alguien actualice la rama local, el repo seguirá desactualizado.
Que opinas? (lo podemos hacer en otro PR)
There was a problem hiding this comment.
Yo entiendo que lo que queremos es que tengan su branch 3.8 local y remota actualizada. ¿Porqué? Porque luego, cuando hagan una nueva rama para trabajar en un archivo nuevo, queremos que tengan los últimos cambios en 3.8 antes de empezar la nueva rama.
Creo que si la persona está trabajando en traduccion-xxx localmente, en principio, no sería necesario actualizar su rama con la upstream/3.8 ya que no hay otra persona tocado el archivo en el que se está trabajando en esa rama.
Supongo que habrá casos en los que sea necesario. Onda, "Esa palabra ya fue agregada al diccionario en esta otra PR que está mergeada. Tenés que traerte los cambios de upstream/3.8 en tu branch local" --pero no creo que sea muy frecuente. No lo sé. De cualquier manera, me parece bien de agregar otra pregunta explicando eso si es necesario.
|
Gracias por aclarar estos puntos acá, dejé un detallito (un espacio) y otro comentario que me surgió cuando leí de nuevo los FAQ. |
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
I added two more questions that appeared some times in the Telegram channel. I think it's good to have them documented here.