Ajuste '//' por 'división entera a la baja' by jcaalzago · Pull Request #2800 · python/python-docs-es · GitHub
Skip to content

Ajuste '//' por 'división entera a la baja' - #2800

Merged
cacrespo merged 7 commits into
python:3.12from
jcaalzago:division_a_la_baja_2
Jan 28, 2024
Merged

Ajuste '//' por 'división entera a la baja'#2800
cacrespo merged 7 commits into
python:3.12from
jcaalzago:division_a_la_baja_2

Conversation

@jcaalzago

Copy link
Copy Markdown
Contributor

Part of #2797

@github-actions

Copy link
Copy Markdown

@jcaalzago
jcaalzago force-pushed the division_a_la_baja_2 branch from 0b7867d to 4e024b9 Compare January 22, 2024 09:08
Comment thread library/stdtypes.po Outdated
#: ../Doc/library/stdtypes.rst:288
msgid "remainder of ``x / y``"
msgstr "resto o residuo de ``x / y``"
msgstr "resto, residuo o módulo de ``x // y``"

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.

Este cambio no me convence. Acá no refiere a floor division. Deberiamos dejar algo así:

Suggested change

¿por qué cambiarlos?

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.

He agregado el término "módulo", porque en gran parte de la documentación se refiere a la operación como módulo y puede que haya gente que donde se menciona sólo cómo módulo, no sepa a qué se está refiriendo.
Ya lo que me digas, lo puedo dejar como estaba.
Espero tu respuesta para hacer el cambio o no.

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.

Además de lo de 'módulo' también está el cambio en "/" por "//".
Si no te molesta, prefiero dejarlo como te sugerí. Y en todo caso levantar el ajuste en la documentación original en inglés.

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.

ok, hago el cambio, ¿sabes cómo se hace para solicitar el cambio en la original en inglés?
Gracias

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.

¿sabes cómo se hace para solicitar el cambio en la original en inglés?

En ese caso hay que hacer un PR a https://github.com/python/cpython
Te recomiendo que revises algunos issues similares (que refieran a files en /Doc) y veas cómo es el proceso para reportar y sugerir un cambio en los textos. De ese modo, le facilitás mucho el trabajo a quién revise el ajuste.

Algunas personas de nuestro grupo de Telegram ya reportaron cambios en los originales en inglés. Tal vez podrías mencionarlo y recibir feedback allí también.

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.

Muchas gracias por la respuesta, ¡máquina!

@jcaalzago
jcaalzago requested a review from cacrespo January 27, 2024 18:04
@cacrespo
cacrespo merged commit 4717496 into python:3.12 Jan 28, 2024
@jcaalzago
jcaalzago deleted the division_a_la_baja_2 branch January 28, 2024 16:54
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.

2 participants