traducir compartiendo módulos al español

[forgondolin]

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?

Issue #165

¿Cuál es el nuevo comportamiento?

[CLAassistant]

All committers have signed the CLA.

[gustavguez]

Hola @forgondolin, muchas gracias por tu aporte, no te olvides de asignar el issue correspondiente al PR, así evitamos que se solapen PRs. Saludos!

[forgondolin]

Hola @forgondolin, muchas gracias por tu aporte, no te olvides de asignar el issue correspondiente al PR, así evitamos que se solapen PRs. Saludos!

Gracias.
Como debo hacer esto?

[forgondolin]

Leí un poco de la guía de colaboradores, pero no sé cómo terminar mi PR

[AntonioCardenas]

Leí un poco de la guía de colaboradores, pero no sé cómo terminar mi PR

Hola @forgondolin editas y agregas fix y el número de issue ejemplo este issue es el #165 y tambien deberias de incluir el archivo en inglés que en tu caso seria crear el archivo sharing-ngmodules.en.md y poner la docu en inglés muchisimas gracias por tu contribución @forgondolin

[gustavguez]

Te ayudé @forgondolin asignando el issue al PR, no te olvides de firmar el CLA

[forgondolin]

Me he "commited" si el nombre equivocado al principio, ahora me he "commited"con "forgondolin", ¿qué puedo hacer para solucionarlo?

[Splaktar]

@forgondolin Necesita usar rebase interactivo (o una buena interfaz de usuario de GitHub como en Webstorm) para aplastar / actualizar sus commits y luego force push de sus commits después de que se elimine el commit de "Kaleb".

Otra opción es crear una nueva rama, aplicar los cambios allí, commit con el usuario correcto y luego force push de esa rama hacia esta rama (forgondolin:feature/sharing-ngmodules-es). Eso actualizará este PR.

Si eso no funciona, puede impulsar su nueva sucursal y crear un nuevo PR.

[gustavguez]

@forgondolin dado que este trabajo está en un fork, dependemos de usted para solucionar este inconveniente, esperaremos 2 semanas desde este comentario, y si no hay respuestas re asignaremos el issue correspondiente. Muchas gracias por colaborar y esperamos que puedas solucionar el problema!

Saludos!

[forgondolin]

hola mil perdones por la demora, terminé olvidándome de esta firma pendiente.
Creo que todo está bien ahora, ¿no?

[Splaktar]

Parece que se resolvieron los problemas con los commits desde cuentas diferentes.

Pero no has seguido los pasos de #235 (comment).

Además, el nombre del archivo debe permanecer igual, sin -es.

[forgondolin]

Hola, modificaciones realizadas. Gracias

[dalejodc]

¡Muchas gracias por tu esfuerzo en esta contribución @forgondolin! 🚀

He hecho algunas observaciones con la traducción. Deben de estar escritas en forma de "tu" y no de "usted". Te sugeriría que puedas hacer commit suggestion en las observaciones, de esta forma no tienes que hacer los cambios manuales.

Creería que eso sería lo último por hacer y luego estamos listos para hacer merge de tu contribución. 👏🏻

[dalejodc]

He ayudado con los cambios de la forma de expresión de "usted" a "tu" en este PR.

Nuevamente gracias por tu aporte @forgondolin.

[gustavguez]

Parece estar todo en orden! :)

[dalejodc]

Parece que CircleCI está fallando por el nombre del commit ca712e8fe9490f3952c70b229c3eb232650c4f5f

[gustavguez]

@forgondolin el commit que dice added: ES page for sharing ng modules
Esta rompiendo el CI, si no arreglamos esto no podemos mergear, disculpa los tiempos de demora, este post puede ayudarte, puedes escribirme y lo hacemos junto hay problema. o Darme permiso a tu fork y te ayudo: https://stackoverflow.com/questions/6942979/add-changes-to-commits-in-the-middle-in-git#:~:text=The%20short%20answer%20is%20that,save%20and%20exit%20the%20editor.