Skip to content

Traducción del archivo library/copyreg#1135

Merged
cmaureir merged 6 commits into
python:3.8from
Xaraxx:library-copyreg
Dec 27, 2020
Merged

Traducción del archivo library/copyreg#1135
cmaureir merged 6 commits into
python:3.8from
Xaraxx:library-copyreg

Conversation

@Xaraxx

@Xaraxx Xaraxx commented Nov 3, 2020

Copy link
Copy Markdown
Contributor

Closes #824

@cmaureir cmaureir left a comment

Copy link
Copy Markdown
Collaborator

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'.

Comment thread .gitignore Outdated
@@ -1 +1,2 @@
*.mo
/env No newline at end of file

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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 👍

Comment thread library/copyreg.po Outdated
Comment thread library/copyreg.po Outdated
Comment thread library/copyreg.po Outdated
Comment thread library/copyreg.po Outdated
Comment thread library/copyreg.po Outdated
Comment thread library/copyreg.po Outdated
Comment thread library/copyreg.po Outdated
Comment thread library/copyreg.po Outdated
Comment thread library/copyreg.po Outdated
@cmaureir

Copy link
Copy Markdown
Collaborator

@Xaraxx necesitas ayudara para continuar con este PR?

Comment thread .gitignore Outdated
@Xaraxx

Xaraxx commented Dec 23, 2020

Copy link
Copy Markdown
Contributor Author

@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
Copy Markdown
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