Skip to content

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

Merged
merged 6 commits into from
Dec 27, 2020
Merged

Conversation

Xaraxx
Copy link
Contributor

@Xaraxx Xaraxx commented Nov 3, 2020

Closes #824

Copy link
Collaborator

@cmaureir cmaureir left a 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
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
/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.

Copy link
Contributor Author

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

Copy link
Contributor Author

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 👍

@cmaureir
Copy link
Collaborator

@Xaraxx necesitas ayudara para continuar con este PR?

@Xaraxx
Copy link
Contributor Author

Xaraxx commented Dec 23, 2020

@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

@cmaureir cmaureir merged commit 0de6cd5 into python:3.8 Dec 27, 2020
@cmaureir
Copy link
Collaborator

No te preocupes @Xaraxx este PR ya estaba listo. Muchas gracias! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Translate 'library/copyreg.po'
2 participants