Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Methods of primitives#105

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed

Conversation

joaquinelio
Copy link
Member

@joaquineliojoaquinelio commentedSep 29, 2019
edited
Loading

First PR ever...

Last commit, basado en la versión EN que corrije una explicación. Y algunas frases más fluidas.

Bienvenidos observadores.
Algunas dudas con hasta dónde traducir ¿hay glosario?

Estoy traduciendo de un fork,
pero para probar instalé el server local... y tiene su propio es.javascript.info, algo engorroso copiar y pegar ¿hay manera mejor?

@CLAassistant
Copy link

CLAassistant commentedSep 29, 2019
edited
Loading

CLA assistant check
All committers have signed the CLA.

@joaquinelio
Copy link
MemberAuthor

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.

@joaquinelio
Copy link
MemberAuthor

@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.

@11joselu
Copy link
Contributor

11joselu commentedNov 25, 2019
edited
Loading

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 branchgit reset --hard upstream/master# Volvemos a recuperar nuestro origingit 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 mastergit checkout master# Creamos otra rama para la tarea de numbersgit checkout -b number

primitives-methods: En esta rama irá todo los relacionado aMethos 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 ramaprimitives-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 haciendocherry-pick's.

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

# Commit con mensaje: 1-5-1git cherry-pick 792e6579816a3ec1f88cb738de8bfb3394d36e36# Commit con el mensaje: 1.5.1 +task+solgit 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 commit253be46 re-traduce lo que habiamos hecho, por tanto, repite los cambios aplicados en el siguiente commit:ba637d1

Por tanto, el commit con el mensaje1 5 1 t5 que apunta a253be46 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 ramanumber

git checkout -b number# Seleccionamos el único commit referente a ese trabajogit cherry-pick 0242104e2b559dabfd3cb71c56641eaa48d76d5e# Subimos la ramagit 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 unpull 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

@joaquinelio
Copy link
MemberAuthor

@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.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@joaquinelio@CLAassistant@11joselu@javascript-translate-bot

[8]ページ先頭

©2009-2025 Movatter.jp