-
Notifications
You must be signed in to change notification settings - Fork 380
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 archivo reference/introduction #2716
Conversation
@JsMelix gracias por la contribución. Podrías modificar la descripción de este PR para que diga "Closes #nnnn" como de explica en el paso 8 de https://python-docs-es.readthedocs.io/es/3.12/CONTRIBUTING.html#paso-2-comienza-a-traducir? Así quedea todo linkeado con el issue original, el ual se cierra automáticamente cuando esté PR se mergee. Gracias! |
¿Por qué lo cerraste @JsMelix ? |
sorry, :) ya esta |
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.
Closes #2716
@JsMelix gracias por actualizar la descripción del PR. Sin embargo ahora hay tests en CI que aún están fallando. Fíjate en los mensajes de error, y por favor corrige lo que sea necesario, así haces la tarea de revisión más fácil. Puedes ver en https://python-docs-es.readthedocs.io/es/3.12/reviewers-guide.html algunos errores comunes de los distintos chequeos que se realizan. |
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.
Ahí van algunos comentarios, gracias por la contribución!
@JsMelix no estoy seguro, pero después de leer tus respuestas tengo la impresión (quizás incorrecta) de que tú crees que mis comentarios son más que nada informativos, y que no falta nada que hacer. Desafortunadamente el proceso no ha terminado, y la idea es que vayas incorporando los comentarios que te la gente va dejando como parte de la revisión. Cuando ya creamos que está todo bien entonces marcaremos el PR como "listo" y lo mergeamos 👍. Como dices tú, se aprende de los errores, pero dentro de esta misma contribución 😄 |
Closes #2646 |
All entries translated, horray! 🎉 |
Co-authored-by: rtobar <rtobarc@gmail.com>
Co-authored-by: rtobar <rtobarc@gmail.com>
Co-authored-by: rtobar <rtobarc@gmail.com>
Co-authored-by: rtobar <rtobarc@gmail.com>
…ix/python-docs-es into traduccion-introduction.po
…ix/python-docs-es into traduccion-introduction.po
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.
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.
Ooops quedaron algunos términos que no pasan el cspell.
@JsMelix Tendrías que agregar dentro del directorio dictionaries
un archivo llamado reference_introduction.txt
con el siguiente contenido:
chose
clonadora
English
stackless
Ya realicé los cambios mencionados :D |
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.
Ahora sí, pasaron los checks 🙌
Closes #2646