GeezerVison

All committers have signed the CLA.

Im pretty lost
added commit based on newer EN source
how to update PR ?
commit amend / push is enough?

Too many commits, conflicts, head, fatal err, merge pending, dirty index, git reset, merge abort, too crazy.

But they all are the same.

@lizzie136
Viendo este PR parece que produje mucha suciedad,
Las varias guías de GIT no me alcanzaron para entender la mecánica.
¿Demasiados commit?
¿Debí cerrar el PR (¿se puede?) y hacer otro?
¿Se puede reemplazar varios commit por uno solo (o PR nuevo)?
¿Puedo mantener 2 PR separados (no veo cómo) ? O es personal. Porque con el úlltimo commit el título del PR quedó desenfocado.
Ahora terminé un tema entero, commit/push y apareció acá, la verdad me siento perdido.
¿Hay algo que deba hacer para que esto no se vea tan desprolijo? Porque no quiero agregarte tareas de más.
¿Es este el lugar (comments PR) para discutir singularidades de una determinada traducción?
Por ej, casi eliminé un párrafo y quería poner el porqué o dejar el tema abierto consejos
Cuando requieran cambios, ¿alcanza con commit/push?
Lo mismo para nuevas traducciones como la de ahora, ¿también commit/push, y aparece acá?

¡UF! Confío, CONFÍO en que las horas invertidas valgan más para USTEDES reviewers y el proyecto que las incomodidades e ineficiencias que produzca por mi inexperiencia.
Si no es así, por favor no te lo guardes, decímelo (del porteño, traducido: "dímelo").

Saludos y felicitaciones por tu labor.
Elio

Ups... ¿qué significa el botón Close and comment? Lo busqué... ¿acaso es tan fácil cerrar un PR? No voy a probar.

Hey, que tál @joaquinelio ?

Vamos a intentar solucionar esto.

Primero que nada, vamos a separar las cosas (dividirlo en dos ramas diferentes para que sea mas fácil de llevar).

Primero que nada vamos a situarnos en master (tu fork) y guardaremos todas las referencias de tus commits.

git log --pretty=oneline

# Output (Ordernador de lo más nuevo a lo más antiguo)
# 0242104e2b559dabfd3cb71c56641eaa48d76d5e 1-05-2 Number, +all the tasks+sol, From EN site
# 07b91ef033a6bc4cb336c56355457ce3e7de1cdb 1 5 1 amend based on EN ver
# 253be4619ebc6763a35409e6b4e7201414e6bedc 1 5 1 t5
# f7dd927c6cfaab2772e5b164cac375f4b91dabe2 Fixes bsed on EN version
# ba637d153d70a3b442750dfd172e26abc432aa8d 1.5.1 +task+sol
# 792e6579816a3ec1f88cb738de8bfb3394d36e36 1-5-1

Ahora restauraremos la rama master a la raíz del proyecto original:

NOTA: no subas nada a tu repositorio tras los siguientes comandos

# Add the remote, call it "upstream":

git remote add upstream https://github.com/javascript-tutorial/es.javascript.info.git

# Fetch all the branches of that remote into remote-tracking branches,
# such as upstream/master:

git fetch upstream

# Make sure that you're on your master branch:

git checkout master

# TAKE CARE! 
# This would delete every commit you made inside master branch

git reset --hard upstream/master 
# Volvemos a recuperar nuestro origin
git remote set-url origin https://github.com/joaquinelio/es.javascript.info.git

Una vez reseteado todo, vamosos a crear dos ramas:

git checkout -b primitives-methods
# Volvemos a master
git checkout master
# Creamos otra rama para la tarea de numbers
git checkout -b number

primitives-methods: En esta rama irá todo los relacionado a Methos of primitives

number: En esta rama irá todo los relacionado a [numbers] (https://github.com/javascript-tutorial/es.javascript.info/tree/master/1-js/05-data-types/02-number).

¡Bien ya estamos listos para restaurar tus commits!

Trabaremos con la primera tarea, por tanto nos situamos en la rama primitives-methods

git checkout  primitives-methods

Ahora meteremos en esa rama tu commits que hace referencia a las traducciones de ese bloque.

Como fuimos precavidos nos guardamos los commits anteriormente, ahora solo lo insertaremos haciendo cherry-pick's.

¡Recuerda el orden cronológico de tus commits, por tanto tenemos que ir de abajo para arriba!

# Commit con mensaje: 1-5-1
git cherry-pick 792e6579816a3ec1f88cb738de8bfb3394d36e36

# Commit con el mensaje: 1.5.1 +task+sol
git cherry-pick ba637d153d70a3b442750dfd172e26abc432aa8d

# Ojo con este commit, eliminas muchas cosas de la versión de inglés y no entiendo por qué. Por tanto habría que hacer una revisión si es el resultado correcto.
git cherry-pick f7dd927c6cfaab2772e5b164cac375f4b91dabe2

Paremos un momento aquí. El siguiente commit 253be46 re-traduce lo que habiamos hecho, por tanto, repite los cambios aplicados en el siguiente commit: ba637d1

Por tanto, el commit con el mensaje 1 5 1 t5 que apunta a 253be46 es totalmente innecesario, por ello, pasamos de introducirlo...

Opcional: el último commit que hace referencia a la primera parte, añade y quita espacios, por tanto, eres libre de meterlo si deseas. Te dejo adivinar ¿Cómo lo hariamos?.

git... 

Bien hemos terminado con esa rama, ahora toca subirlo al repositorio y lanzar el MR!

git push --set-upstream origin primitives-methods

Al hacer push, te dará un url para crear el PR o sino si vas a tu repositorio, el propio github te dirá para crear un MR al fork :)

Ahora queda la segunda parte

Nos situamos en la rama number

git checkout -b number

# Seleccionamos el único commit referente a ese trabajo
git cherry-pick 0242104e2b559dabfd3cb71c56641eaa48d76d5e
# Subimos la rama
git push --set-upstream origin number

Y repetimos el MR!

Ahora bien, eres libre de cerrar este PR ya que no te servirá más :)

Esto es todo, así de "fácil" solucionamos tu problema, ahora bien te recomiendo que por cada tarea crees una rama partiendo siempre de master original (fork)

Espero haberte ayudado :)

pd: Recuerda la nota de no subir los cambios de master? Puedes hacer un pull y volverías a tener tus commits allí, yo te recomiento no volver a traertelo, sino actualizar master con la versión original.

¿Cómo hacer esto?

Si no has hecho pull aún, actualizamos master de manera "forzosa"

git checkout master && git push -f

@11joselu
TLDR: Gracias, aunque no veas aún el arreglo más me valió el schooling.

¡UF!
Qué arduo que es esto.
Pero ¡gracias! Lo tuyo es más un curso de lo que GIT es (especialmente el trabajo en ramas) y había obviado completamente.
Así como hice un proyecto vanila JS para aprender, debí usar la terminal GIT en vez de usar la interfaz de VS CODE que te oculta lo que está haciendo (no tenía idea casi).

Cada paso es instructivo... Aunque no hice ninguno aún. Me he peleado con mi W10 (en todo) y ahora estoy en peleas frescas con el Deepin (que elegí por lindo) alguna vez lograré el entorno andando.

Como curiosidad en un comentario sobre uno de los commit: sobre muchas cosas que "eliminé" del inglés. Había dos errores en un tema. Fiel al espíritu, lo traduje como estaba pero me fui al EN para ver si podía notarlo sin un PR (como observador, jeje). Sorpresa, ya estaba corregido y mejorado, retraduje sobre ese. La versión inglés que tenemos en ES es en algunas cosas muy vieja y por eso la supuestas inconsistencias.
Habría que recomendar, al quien haga traducción, que compare fecha con el fork EN.

No quisiera perder las horas de traducción (ok, imputable al aprendizaje del entorno), So:
I'll be back
Si llego antes del ACV.

Salud.