Skip to content

Varias correcciones en el módulo time #1831

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 12 commits into from
Aug 4, 2022
Merged

Varias correcciones en el módulo time #1831

merged 12 commits into from
Aug 4, 2022

Conversation

gio8tisu
Copy link
Contributor

@gio8tisu gio8tisu commented Aug 3, 2022

La explicación de cada cambio está disponible en el commit correspondiente.
Recomiendo hacer squash-merge para no ensuciar la historia con commits muy relacionados.
Cualquier sugerencia, estoy dispuesto a aceptar cambios.

gio8tisu and others added 8 commits August 3, 2022 11:22
'Time access' se refiere a 'acceso a (funcionalidades relacionadas con)
tiempo' no a 'tiempo de acceso'.
El orden de las palabras no era del todo correcto.
* "mucho en el futuro" por "muy adelante en el futuro"
* Varios articulos
* Palabras sin traducir
@cmaureir
Copy link
Collaborator

cmaureir commented Aug 3, 2022

Buenísimo @gio8tisu 🎉 Muchas gracias! a ver si el CI pasa y lo mergeamos ;)

Copy link
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.

Súper buen trabajo! Dejé un solo comentario, ojalá no necesite un powrap adicional.

@cmaureir
Copy link
Collaborator

cmaureir commented Aug 3, 2022

Aparte del comentario de Rodrigo, falta pasar un powrap al archivo: powrap library/time.po y agregar los cambios y hacer otro commit 👍

@gio8tisu
Copy link
Contributor Author

gio8tisu commented Aug 3, 2022

Hola, muchas gracias por los comentarios. He aplicado los cambios reescribiendo la historia de commits.

@cmaureir
Copy link
Collaborator

cmaureir commented Aug 3, 2022

Hola @gio8tisu 👋
Sigue faltando que ejecutes powrap sobre el archivo que modificaste.
Por cierto, no te preocupes en hacer force push ni nada, al final hacemos Squash + Merge asi que la historia se transformará en 1 commit en la rama principal.

"de la época o mucho en el futuro. El punto de corte en el futuro está "
"determinado por la biblioteca C; para sistemas de 32 bits, normalmente es en "
"2038."
"de la época o muy adelante en el futuro. El punto de corte en el futuro "
Copy link
Contributor

Choose a reason for hiding this comment

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

Esto ya fue traducido así, por lo que capaz que ya se discutió, pero época me parece que podría dejarse como epoch, ya que así es como se lo conoce a ese instante de tiempo específico.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Personalmente, yo también preferiría epoch, al principio se da la definición de la palabra así que no le veo inconveniente. Pero no quise tomar esa decisión. Si estáis de acuerdo puedo sustituir todas las ocurrencias en el módulo.

@gio8tisu
Copy link
Contributor Author

gio8tisu commented Aug 4, 2022

Hola @gio8tisu wave Sigue faltando que ejecutes powrap sobre el archivo que modificaste

@cmaureir powrap en local no me aplica ningún cambio.

@cmaureir
Copy link
Collaborator

cmaureir commented Aug 4, 2022

Hola @gio8tisu wave Sigue faltando que ejecutes powrap sobre el archivo que modificaste

@cmaureir powrap en local no me aplica ningún cambio.

Estás en windows?

@gio8tisu
Copy link
Contributor Author

gio8tisu commented Aug 4, 2022

Hola @gio8tisu wave Sigue faltando que ejecutes powrap sobre el archivo que modificaste

@cmaureir powrap en local no me aplica ningún cambio.

Estás en windows?

Nop, Ubuntu 22. Lo ejecuto desde el virtualenv creado con make venv.

@gio8tisu
Copy link
Contributor Author

gio8tisu commented Aug 4, 2022

Curiosamente si ejecuto powrap sobre todos los ficheros (con make wrap) me cambia muchos ficheros, pero no library/time.po.

@rtobar
Copy link
Collaborator

rtobar commented Aug 4, 2022

Curiosamente si ejecuto powrap sobre todos los ficheros (con make wrap) me cambia muchos ficheros, pero no library/time.po.

Yo veo exactamente el mismo efecto en mi entorno local efectivamente, también con Ubuntu 22.04, python 3.10.4, powrap 1.0.0.

Leyendo un poco, powrap usa msgcat (parte del paquete gettext de GNU) por debajo, el cual tiene una opción para setear el ancho que tienen que tener los mensajes, pero que powrap nunca setea. Leyendo el código de msgcat, si no se especifica un ancho se usa el valor por defecto, que es 79 -- y que cambió a ese valor el año 95 o 96... asi que por ahí no va la cosa :P

Voy a ver si puedo entender por qué se produce la diferencia para poder solucionar el tema de fondo.

@gio8tisu
Copy link
Contributor Author

gio8tisu commented Aug 4, 2022

He logrado reproducirlo usando act. Añadiendo el flag --diff al workflow he podido ver que lineas eran conflictivas y las he editado, añado los cambios en un nuevo commit.

@cmaureir cmaureir merged commit ef2d567 into python:3.10 Aug 4, 2022
@cmaureir
Copy link
Collaborator

cmaureir commented Aug 4, 2022

Muchas gracias @gio8tisu 🎉 🎉

@gio8tisu gio8tisu deleted the fix-time branch August 4, 2022 22:31
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