docs: start-routing Traducido al Español#236
docs: start-routing Traducido al Español#236Splaktar merged 15 commits intoangular-hispano:masterfrom
Conversation
|
|
|
Muchas gracias por tu contribución pero veo que omitiste el issue template es muy importante que lo uses ya que esto ayuda quienes revisan el código a saber que testear y qué issue es ver |
dalejodc
left a comment
dalejodc
left a comment
There was a problem hiding this comment.
¡Muchas gracias por tu aporte con este PR @mferreiras ! 🚀
He hecho algunas observaciones con la forma de expresión de "usted", debe cambiarse a "tú". También he hecho algunas sugerencias en las traducciones. Si estás de acuerdo con ellas es mucho más facil hacer commit suggestion desde acá en UI de Github en lugar de cambiarlo desde tu código.
También debes de agregar el archivo original con el contenido en inglés; en tu caso el start-routing.en.md.
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
dalejodc
left a comment
dalejodc
left a comment
There was a problem hiding this comment.
Genial, muchas gracias por tu aporte @mferreiras 👏. En la primera revisión omití dos observaciones, que deben de manejarse. Puedes hacer nuevamente commit suggestion desde acá.
Y sugiero que continuemos con éste PR y no el nuevo que abriste. Así que lo mejor sería que agregaras desde tu local el archivo original en inglés y luego continuamos con el merge de tu contribución.
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
De acuerdo @dalejodc |
|
Listo @mferreiras . Únicamente falta el archivo original renombrado a start-routing.en.md |
Splaktar
changed the title
Listo he realizado el commit 👍. @dalejodc |
gustavguez
left a comment
gustavguez
left a comment
There was a problem hiding this comment.
El último commit dió fail en el lint, por la convención de los commits, pero el resto parece estar bien.
6 participants
Lista de Verificación del PR
Comprueba si tu PR cumple los siguientes requisitos:
El mensaje de commit esta conforme con nuestras reglas
Probe los cambios que agregué (arreglo de bugs / funcionalidades)
Revisé previamente las traducciones o cambios de contenido
Consulté el diccionario de términos en español
He creado dos archivos con la extensión correspondiente(.en.md para el archivo en inglés y .md para el Archivo en español)
La PR fue enlazada con el issue correspondiente
Tipo de PR
¿Qué tipo de cambio introduce este PR?
Bugfix
Funcionalidad
Actualización de el estilo del código (formato, variables locales)
Refactorización (no cambios en la funcionalidad, no cambios en el api)
Cambios relacionados al build
Cambios relacionados al CI (Integración continua)
Cambios en el contenido de la documentación
Cambios en la aplicación / infraestructura de angular.io
Otro... Por favor describe la:
¿Cuál es el comportamiento actual?
start-routing.md se encuentra en inglés
¿Cuál es el nuevo comportamiento?
Los archivos se ha traducido al español
feat(lang): #118