Skip to content

Actualizar a 3.10.2#1821

Closed
cmaureir wants to merge 1 commit into
3.10from
3.10.2
Closed

Actualizar a 3.10.2#1821
cmaureir wants to merge 1 commit into
3.10from
3.10.2

Conversation

@cmaureir
Copy link
Copy Markdown
Collaborator

Actualización de 3.10.0 a 3.10.2 para cubrir los pequeños cambios en la documentación.

@cmaureir
Copy link
Copy Markdown
Collaborator Author

@cacrespo @rtobar @humitos @raulcd alguien se ánima a echarle un vistazo para hacer el merge?

@humitos
Copy link
Copy Markdown
Collaborator

humitos commented Jan 25, 2022

@cmaureir lo hacemos en streaming? 😝

Yo no creo que lo pueda mirar pronto. Probablemente, no antes de en 2 semanas 😞 --pero ahí me sumo!

@cmaureir
Copy link
Copy Markdown
Collaborator Author

jajaja para 3.11 hay que hacer un stream si o sí, igual este cambio es "pequeño" en comparación con lo que nos tocó hacer, acá nada se rompió y todo funciona porque solo son algunas modificaciones a algunos archivos 👼

Copy link
Copy Markdown
Collaborator

@rtobar rtobar left a comment

Choose a reason for hiding this comment

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

Revisado hasta lib/importlib.metadata.po

Aprendí de donde vienen algunos de los artefactos que he encontrado en algunos de los archivos .po (traducciones comentadas al final del archivos, especificación de python-format en algunas de las entradas). En general se ve bien, pero hay varias traducciones que se "pierden" porque quedan comentadas al final del archivo.

Para ayudar a reducir un poco el ruido fui haciendo la revisión visual de forma local con git diff -U0 origin/3.10..origin/3.10.2 | grep -v '\(^[-+]#: \)\|\(^@@\)' | less para poder filtrar los cambios en las líneas de los archivos po

Comment thread c-api/init_config.po

#: ../Doc/c-api/init_config.rst:637
msgid "Dump Python refererences?"
#, fuzzy
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.

quizás este puede quedar sin fuzzy, la traducción está buena ya

Comment thread copyright.po
Comment on lines +48 to +49
"Derechos de autor © 1995-2000 Corporation for National Research Initiatives. "
"Todos los derechos reservados."
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.

Aquí se wrapeó la traducción, pero está bien igual supongo? Igual raro, pasaste los archivos por powrap?

Comment thread copyright.po
Comment on lines +55 to +56
"Derechos de autor © 1991-1995 Stichting Mathematisch Centrum. Todos los "
"derechos reservados."
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.

Acá también

Comment thread extending/extending.po
"the members of the Python tuple returned by :meth:`sys.exc_info`. These are "
"the exception type, exception instance, and a traceback object. It is "
"important to know about them to understand how errors are passed around."
msgstr ""
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.

Acá se perdió totalmente la traducción

Comment thread extending/extending.po
"Estas garantías no se cumplen cuando utiliza la convención de llamadas de "
"estilo \"antiguo\", que todavía se encuentra en muchos códigos existentes."

#~ msgid ""
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.

Interesante, la traducción que se perdió quedó acá... no sé cómo funcionan las herramientas que se usan para actualizar los .po, pero esto es nuevo para mí. Lo resalto solamente, no sé si está bien o mal.

Comment thread library/ctypes.po
msgid ""
"Raises an :ref:`auditing event <auditing>` ``ctypes.seh_exception`` with "
"argument ``code``."
msgstr ""
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.

ditto

Comment thread library/ctypes.po
msgid ""
"Raises an :ref:`auditing event <auditing>` ``ctypes.call_function`` with "
"arguments ``func_pointer``, ``arguments``."
msgstr ""
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.

ditto

Comment thread library/datetime.po
#: ../Doc/library/datetime.rst:31
#, fuzzy
msgid "Module :mod:`zoneinfo`"
msgstr "Módulo :mod:`time`"
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.

De nuevo pareciera ser que se está agreando ya una traducción (incorrecta)

Comment thread library/enum.po
"Private names will be normal attributes in Python 3.10 instead of either an "
"error or a member (depending on if the name ends with an underscore). Using "
"these names in 3.9 will issue a :exc:`DeprecationWarning`."
":ref:`Private names <private-name-mangling>` will be normal attributes in "
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.

Aquí nuevamente se perdió la traducción

Comment thread library/functools.po
"cached separately. If *typed* is false, the implementation will usually "
"regard them as equivalent calls and only cache a single result. (Some types "
"such as *str* and *int* may be cached separately even when *typed* is false.)"
msgstr ""
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.

se perdió la traducción

@humitos
Copy link
Copy Markdown
Collaborator

humitos commented Jan 26, 2022

Me acabo de dar cuenta que esto es una PR y que el trabajo de actualizar ya está hecho. Solo estabas pidiendo ayuda con la revisión 🤦🏼

¿Seguiste los pasos de https://python-docs-es.readthedocs.io/es/3.10/upgrade-python-version.html con la actualización que habíamos hecho cuando hicimos el streaming? Si es así, es muy probable que esté todo bien 😉

Por lo general para este tipo de trabajo, creo que es más fácil hacer una doble validación de los comandos ejecutados (pair programming como hicimos la otra vez) que revisar una PR como esta de 58k líneas. Yo creo que esta review es imposible y probablemente una pérdida de tiempo siendo que los cambios son generados por un software en el que confiamos 100% --por eso, para mí, el único lugar dónde puede haber un fallo, es en los comandos ejecutados.

@cmaureir
Copy link
Copy Markdown
Collaborator Author

Gracias @rtobar y disculpa por no explicar como iba el tema de las actualizaciones.
Si una entrada cambia un poco su contenido, la traducción 'se pierde' completamente, y pasa a estar como una entrada comentada al final del archivo y esa queda con un msgstr vacío. Principalmente porque el merge es más complicado, asi que no te asustes si ves que se quitan traducciones :P

@cmaureir
Copy link
Copy Markdown
Collaborator Author

¿Seguiste los pasos de https://python-docs-es.readthedocs.io/es/3.10/upgrade-python-version.html con la actualización que habíamos hecho cuando hicimos el streaming? Si es así, es muy probable que esté todo bien wink

Si, de hecho seguí esa guía más los comentarios que queríamos agregar (que aún no se hace), e hice una construcción local antes para verificar que todo estaba OK.

De hecho ya tengo lista la info para cuando esto se haga merge, podamos crear más issues

"filename";"untranslated";"total";"fuzzy"
"distutils/sourcedist.po";1;74;2
"howto/logging-cookbook.po";4;237;0
"reference/datamodel.po";23;507;45
"reference/expressions.po";1;328;6
"reference/executionmodel.po";11;49;0
"library/logging.config.po";1;127;1
"library/os.path.po";1;69;0
"library/asyncio-task.po";1;184;0
"library/dataclasses.po";2;111;5
"library/functools.po";2;84;11
"library/enum.po";3;189;0
"library/multiprocessing.po";2;518;3
"library/http.server.po";1;89;4
"library/datetime.po";2;563;5
"library/sqlite3.po";12;219;6
"library/os.po";1;841;9
"library/importlib.po";9;343;0
"library/statistics.po";1;172;0
"library/calendar.po";1;70;1
"library/exceptions.po";2;148;2
"library/urllib.parse.po";1;149;1
"library/logging.po";1;324;4
"library/timeit.po";1;53;0
"library/sys.po";1;401;3
"library/typing.po";23;359;4
"library/collections.abc.po";1;119;0
"library/zipimport.po";1;39;0
"library/asyncio-eventloop.po";2;353;1
"library/logging.handlers.po";3;262;0
"library/asyncio-future.po";1;61;0
"library/stdtypes.po";23;994;31
"library/functions.po";1;423;5
"tutorial/introduction.po";1;74;0
"faq/library.po";2;144;0
"extending/newtypes.po";1;89;1
"extending/extending.po";1;159;0
"c-api/typeobj.po";1;598;3

Los mayores cambios generaron como 23 entradas en 4 archivos más o menos
el resto son solo un par de entradas que hay que volver a traducir.

El módulo cpython tiene el último commit de ayer en 3.10

Realmente como el CI pasó sin problema, era más para que alguien se animara a aprobar el cambio y hacíamos merge, y no una revisión detallada de cada archivo, pues al haber algo incorrecto, sphinx se iba a quejar :P

@humitos
Copy link
Copy Markdown
Collaborator

humitos commented Jan 30, 2022

Si, de hecho seguí esa guía más los comentarios que queríamos agregar (que aún no se hace), e hice una construcción local antes para verificar que todo estaba OK.

Sería bueno si podemos hacer este trabajo así queda actualizado. Si querés pasame las notas y yo lo actualizo 😉

Realmente como el CI pasó sin problema, era más para que alguien se animara a aprobar el cambio y hacíamos merge, y no una revisión detallada de cada archivo, pues al haber algo incorrecto, sphinx se iba a quejar :P

Me parece perfecto!

@cacrespo
Copy link
Copy Markdown
Collaborator

Esto quedó desactualizado ¿cierto? Valdrá la pena intentar impactar los ajustes @cmaureir ?

@humitos
Copy link
Copy Markdown
Collaborator

humitos commented Oct 26, 2022

Yo creo que lo mejor es cerrar este PR y enfocarse en 3.11 😄

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.

4 participants