-
Notifications
You must be signed in to change notification settings - Fork 85
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
Traducido observables.md a español fix #214 #258
base: master
Are you sure you want to change the base?
Conversation
Se mantiene el uso de streams para referirse a flujos ya que se conocen más por su nombre en inglés, incluso en la comunidad hispana.
@jaov Gracias por colaborar! debes firmar el CLA y debes asignar el issue correspondiente en el PR, antes que comencemos a revisar tu PR. Saludos! |
Arreglados algunos errores ortográficos y de redacción en la traducción.
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.
Hola @jaov, deben de incluir el archivo original con la extensión en.md para poder revisar tu PR. saludos!
@jaov sigues trabajando con el archivo? me gustaría contribuir |
El archivo se encuentra en esta URL dentro del proyecto. ¿Es necesario que lo suba aparte? |
Si @jaov, el archivo original se debe de incluir con extensión .en.md. En este caso el PR debe de incluir el archivo observables.md (el que está actualmente en el PR) y el archivo observables.en.md que es el archivo original en inglés (el que hace falta en el PR). |
Se mantiene el uso de streams para referirse a flujos ya que se conocen más por su nombre en inglés, incluso en la comunidad hispana.
Lista de Verificación del PR
Comprueba si tu PR cumple los siguientes requisitos:
Resolviendo Issue #214
Tipo de PR
¿Qué tipo de cambio introduce este PR?
¿Cuál es el comportamiento actual?
Archivo está en inglés
¿Cuál es el nuevo comportamiento?
Archivo de inglés traducido al español