-
Notifications
You must be signed in to change notification settings - Fork 396
Traduccion library/tracemalloc #935
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
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.
Excelente trabajo @bleucode . Felicitaciones!
Te dejé algunos comentarios:
- Te propuse traducir "frame" que lo habías dejado en inglés por "cuadro"
- Te marqué las etiquetas
:term:
,:class:
,:func:
,:envvar:
, etc que no fueron incluidas en la traducción. No marqué todas porque no quería llenarte de comentarios sobre esto, sólo apunté las que encontré faltantes hasta la mitad del documento. Creo que a partir de ahí se entiende la idea 😃
La idea es que si tenemos, por ejemplo:
:func:`tracemalloc.start`
en el texto original, lo dejamos tal cual (incluido el :func:
). Lo mismo sucede con todas las etiquetas similares como :class:
, :mod:
, etc.
Ese es el motivo por el cual Travis está fallando.
Espero encuentres de utilidad mis comentarios y podamos mergear este PR pronto.
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
@bleucode si vas aceptando y arreglando las sugerencias por tu cuenta y sin usar la interfaz de Github, recuerda ir marcando 'Resolve conversation' en cada una de ellas! así podemos ver si hay algún sugerencias pendientes 👍 |
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
cuando vas a la pestaña "Files Changed" puedes mirar todas las sugerencias y presionar "Add suggestion to batch" y asi hasta que terminas y luego haces 1 solo commit con todos los cambios incluidos, no es necesario hacerlo de a uno. |
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Ay perdón! La próxima pregunto antes, no tenia idea de que era así. Gracias por la aclaración. |
@Bgeninatti gracias por la ayuda y las aclaraciones, la verdad no tenia idea y pensé que no iban las etiquetas, me pongo en modo editar y revisar que estén todas 👍 |
Es solo para hacerte la vida más facil, asi es solo apretar botones :P super rapido a mi parecer:
aún hay un par de palabras que no reconoce pospell, y luego recuerda pasar powrap cuando las corrijas ;) |
Resuelto con etiquetas y cambios en algunas traducciones como "frame".
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
No entiendo porque falla el build :( ayuda @cmaureir @Bgeninatti |
@bleucode Te debería llevar a esta página: https://github.com/python/python-docs-es/pull/935/checks?check_run_id=1256221571
Pero fijate que noto esto de las palabras faltantes:
|
Gracias por la ayuda! Tengo problemas al usar powrap y pospell. Me dijeron que es común en varios usuarios de windows, por lo menos pospell. Voy a investigar como solucionarlo. |
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Arreglando espacios y powrap del PR |
Gracias @bleucode, quedaban solo un par de palabras y powrap asi que lo arreglé para que se pudiera mergear. |
@cmaureir gracias a ti! la verdad estaba bastante perdida y recién veo esta notificación. |
Closes #673