Skip to content

Conversation

@kenliten
Copy link
Contributor

Hola @joaquinelio!
Lamento mucho la demora, tuve muchos problemas lidiando con una distro que no conocía e intenté traducir desde el celular y creé muchos problemas, así que al final, tuve que eliminar mi fork y con el el progreso, para crear un nuevo fork, un nuevo branch y bueno, aquí está el primer artículo.

PeAcE!

@joaquinelio
Copy link
Member

hola!
no es una carrera...
y tus aportes ya sumaron al proyecto

disculpame ahora la crítica,
"importance: " de las task no se traducen, NO SE por qué, es metadata pero no se para que la usa
y la verdad que en ingles queda feo,
pero específicamente (en el readme) pidio que no tradujeramos

 Metadata Some files, usually tasks, have YAML metadata at the top, delimited by ---: importance: 5 --- ... Please don't translate "importance" (and other top metadata). 

@kenliten
Copy link
ContributorAuthor

Oh ok!

Copy link
Member

@joaquineliojoaquinelio left a comment

Choose a reason for hiding this comment

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

typos y sugerencias

No traduzcas "importance" en task porque es metadata que no se para que corno es

sobre lo que puse de aclarar "mousedown", no hace falta, lo resolviste bien y queda claro.

@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@kenliten
Copy link
ContributorAuthor

/done
Si eso era todo, estamos listos!

@joaquinelio
Copy link
Member

Hola capo

No, no era todo. Hay que deshacer los dos cambios
"importance: 5"
no puedo editarlo ni me deja sugerir cambios no se por qué

ya va un mes,
si no aparece segundo reviewer lo reviso de nuevo y -> merge.

@joaquinelio
Copy link
Member

@kenliten
ah, borraste tu branch
pero github aun lo recuerda en nosedonde,
asi que se puede hacer merge igual
y corregir lo de "importance" después

@kenliten
Copy link
ContributorAuthor

Si, lo borre, pense que ya habias hecho el merge, y ahora intento pero no me permite editarlos

@joaquinelio
Copy link
Member

No te permite porque está trabajando con una versión congelada

Voy a hacer merge y un despues PR con más cambios para:

  • arreglar "importance:" (aunque nadie lo extrañe)
  • legibilidad de las lineas mas importantes del principio

@joaquineliojoaquinelio merged commit 7861d07 into javascript-tutorial:masterJan 16, 2021
@javascript-translate-bot

Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@kenliten@joaquinelio@javascript-translate-bot