Skip to content
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: traducción de npm-packages.md #340

Merged
merged 13 commits into from
Apr 27, 2021
Merged

docs: traducción de npm-packages.md #340

merged 13 commits into from
Apr 27, 2021

Conversation

gustavguez
Copy link
Member

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?

Actualmente el archivo npm-packages.md está en Inglés solamente.

¿Cuál es el nuevo comportamiento?

El archivo en Inglés pasa a tener extensión npm-packages.en.md y el archivo npm-packages.md pasa a estar traducido al Español.
Fix #213

@gustavguez gustavguez mentioned this pull request Apr 22, 2021
15 tasks
@gustavguez gustavguez requested a review from a team April 22, 2021 01:37
Copy link
Member

@dalejodc dalejodc left a comment

Choose a reason for hiding this comment

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

Excelente aporte @jLagosCarrera @gustavguez.

Solo tengo algunas sugerencias con algunas tildes que faltan y el uso de palabras con inicial mayúscula.

aio/content/guide/npm-packages.md Outdated Show resolved Hide resolved
aio/content/guide/npm-packages.md Outdated Show resolved Hide resolved
aio/content/guide/npm-packages.md Outdated Show resolved Hide resolved
aio/content/guide/npm-packages.md Outdated Show resolved Hide resolved
aio/content/guide/npm-packages.md Outdated Show resolved Hide resolved
aio/content/guide/npm-packages.md Outdated Show resolved Hide resolved
aio/content/guide/npm-packages.md Outdated Show resolved Hide resolved
aio/content/guide/npm-packages.md Outdated Show resolved Hide resolved
aio/content/guide/npm-packages.md Outdated Show resolved Hide resolved
gustavguez and others added 8 commits April 22, 2021 09:00
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>
@gustavguez gustavguez requested a review from dalejodc April 22, 2021 12:03
@gustavguez
Copy link
Member Author

@dalejodc Muchas gracias por tu review!

Apliqué los cambios.

Saludos! y buen trabajo!

Copy link
Member

@dalejodc dalejodc left a comment

Choose a reason for hiding this comment

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

¡Genial! @gustavguez

Me parece bien.

@gustavguez gustavguez added the Listo y Revisado listo para hacer merge label Apr 24, 2021
aio/content/guide/npm-packages.md Outdated Show resolved Hide resolved
aio/content/guide/npm-packages.md Outdated Show resolved Hide resolved
@Splaktar Splaktar merged commit cf2d40e into master Apr 27, 2021
@Splaktar Splaktar deleted the gustavguez-fix-252 branch April 27, 2021 13:52
@dalejodc dalejodc changed the title Docs traducción de npm-packages.md docs: traducción de npm-packages.md Apr 30, 2021
ealexisaraujo added a commit to ealexisaraujo/angular that referenced this pull request Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Listo y Revisado listo para hacer merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Traducir: guide/npm-packages.md
4 participants