-
Notifications
You must be signed in to change notification settings - Fork 86
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
docs(docs-infra): traducir guide/property-binding.md
(#126)
#421
base: master
Are you sure you want to change the base?
docs(docs-infra): traducir guide/property-binding.md
(#126)
#421
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Buen trabajo y gracias por el esfuerzo cometido.
Creo que algunos cambios son necesarios. Son detalles como intentar de no añadir más palabras de las necesarias para traducir/expresar algo, sugerencias de otras palabras, algunas correcciones de laísmos o cambios en el orden de algunas palabras.
Ninguna red flag pero si consideraría estos cambios necesarios para el merge del PR.
De nuevo, buen trabajo y muchas gracias! Cualquier duda, puedes comentarnos.
usar terminologia apropiada Co-authored-by: Alejandro Lora <[email protected]>
…p ci] usar nombre de manera consistente
…p ci] corrige conjugacion de verbo gracias @alejandrocoding
…p ci] usa menos palabras gracias @alejandrocoding
…p ci] usa sinónimo más apropiado para el concepto Co-authored-by: Alejandro Lora <[email protected]>
…p ci] quita un artículo redundante y mejora la puntuación Co-authored-by: Alejandro Lora <[email protected]>
…p ci] quita un artículo redundante Co-authored-by: Alejandro Lora <[email protected]>
…p ci] reordena el flujo de la oración; mejora la claridad del mensaje Co-authored-by: Alejandro Lora <[email protected]>
…p ci] usa un conector mas apropiado Co-authored-by: Alejandro Lora <[email protected]>
…p ci] usa terminologia apropiada y simplifica la oracion Co-authored-by: Alejandro Lora <[email protected]>
…p ci] simplifica la expresion Co-authored-by: Alejandro Lora <[email protected]>
…p ci] elimina un adjetivo innecesario Co-authored-by: Alejandro Lora <[email protected]>
…p ci] Co-authored-by: Alejandro Lora <[email protected]>
…p ci] elimina adjetivos y preposiciones innecesarias Co-authored-by: Alejandro Lora <[email protected]>
…p ci] corrige frases y palabras mal traducidas Co-authored-by: Alejandro Lora <[email protected]>
…p ci] elimina adjetivo innecesario Co-authored-by: Alejandro Lora <[email protected]>
…p ci] simplifica y mejora la claridad de la oracion Co-authored-by: Alejandro Lora <[email protected]>
…p ci] Co-authored-by: Alejandro Lora <[email protected]>
…p ci] Co-authored-by: Alejandro Lora <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gracias por tu revisión @alejandrocoding
Acepté la mayor parte de tus sugerencias, pero hay un par con las que tengo algo que decir. Traté de exponer mis percepciones de manera clara, espero podamos llegar a acuerdo 🙂
…p ci] usa sinonimo de verbo apropiado Co-authored-by: Alejandro Lora <[email protected]>
…p ci] usa sustantivo apropiado Co-authored-by: Alejandro Lora <[email protected]>
…p ci] mejora frase usando adjetivo cuantitativo Co-authored-by: Alejandro Lora <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Muchas gracias por añadir las sugerencias y comentar el resto, excelente trabajo!
Lista de Verificación del PR
Comprueba si tu PR cumple los siguientes requisitos:
Tipo de PR
¿Qué tipo de cambio introduce este PR?
¿Cuál es el comportamiento actual?
¿Cuál es el nuevo comportamiento?
Se traduce el archivo
guide/property-binding.md
. Este PR pretende cerrar el issue #126.