-
Notifications
You must be signed in to change notification settings - Fork 396
Traducción del archivo library/copyreg #1135
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.
Gracias por la traducción @Xaraxx :) te dejé un par de comentarios:
- Si en el futuro quieres modificar archivos que no estén relacionado con el archivo de traducción y su respectivo diccionario, te recomiendo que puedas primero abrir una issue para que lo podamos discutir.
- Encontré varios espacios dobles en la traducción, por mucho que son indefensos, tenemos que intentar dejar la estructura del texto lo más cercana a la original.
- 'pickling' ha sido discutido antes, y la conclusión es utilizar 'serializado (pickling)' pues no es un serializado único, incluso en Python, donde también hay conceptos como marshaling, que también corresponde a un serializado, por eso es importante dejar entreparéntesis la palabra que nos indique que está relacionado al módulo 'pickle'.
- Cuando hablamos de 'raises and exception' se usa la palabra "lanzar" https://python-docs-es.readthedocs.io/es/3.8/translation-memory.html y lo mismo con "Return" que se debe usar 'retorna' en vez de 'devuelve'.
.gitignore
Outdated
@@ -1 +1,2 @@ | |||
*.mo | |||
/env |
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.
/env |
no es necesario este cambio, el entorno virtual que se crea con el script se llama 'venv' si mal no recuerdo y si creaste uno llamado 'env' está igual de correcto, pero no creo que valga la pena actualizar el .gitignore de todos.
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! lo siento es que tengo ese virtual environment y lo incluí para que no se subiera con mis cambios, en todo caso lo voy a eliminar y a crear en otro lado para no afectar el gitignore de todos
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.
@cmaureir muchas gracias por tus comentarios y ya pronto corrijo el PR 👍
@Xaraxx necesitas ayudara para continuar con este PR? |
@cmaureir te cuento que he estado teniendo algunos problemas técnicos con mi computador pero ya pronto voy a hacer las correciones, lamento mucho la demora |
No te preocupes @Xaraxx este PR ya estaba listo. Muchas gracias! 🎉 |
Closes #824