python/python-docs-esPublic

NotificationsYou must be signed in to change notification settings
Fork396
Star349

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

cmaureir merged 12 commits intopython:3.10fromgio8tisu:fix-time

Aug 4, 2022

Merged

Varias correcciones en el módulo`time`#1831

cmaureir merged 12 commits intopython:3.10fromgio8tisu:fix-time

Aug 4, 2022

Conversation

Copy link

Contributor

gio8tisu commentedAug 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.

gio8tisuand others added8 commits

August 3, 2022 11:22

Corrige título del módulo

67c1647

'Time access' se refiere a 'acceso a (funcionalidades relacionadas con)tiempo' no a 'tiempo de acceso'.

Aclara la descripcion de disponibilidad del módulo

2156ff1

El orden de las palabras no era del todo correcto.

Corrige "segundos bisiestos" por "segundos intercalares"

49590c0

Le da mas consistencia y exactitud (véasehttps://es.wikipedia.org/wiki/Segundo_intercalar).

Corrige errores menores

1ddab1c

* "mucho en el futuro" por "muy adelante en el futuro"* Varios articulos* Palabras sin traducir

Corrige monotónico por monótono

5aa7faf

La palabra correcta es monótono (verhttps://es.wikipedia.org/wiki/Funci%C3%B3n_mon%C3%B3tona).

Corrige "programación" por "planificación"

cc34cad

Creo que es mas correcto (verhttps://es.wikipedia.org/wiki/Planificador).

Corrige "sentido" por "significado"

fbd57a0

Usar "lanzar" para "raise"

007a82b

Copy link

Collaborator

cmaureir commentedAug 3, 2022

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

rtobar requested changes

Aug 3, 2022

View reviewed changes

Copy link

Collaborator

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.

library/time.po OutdatedShow resolvedHide resolved

Copy link

Collaborator

cmaureir commentedAug 3, 2022

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

gio8tisu added2 commits

August 3, 2022 14:59

Corrige "julio" por "juliano"

3896b52

Fix wrapping

650fcb3

Copy link

ContributorAuthor

gio8tisu commentedAug 3, 2022

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

rtobar approved these changes

Aug 3, 2022

View reviewed changes

Copy link

Collaborator

cmaureir commentedAug 3, 2022

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

eamanu approved these changes

Aug 3, 2022

View reviewed changes

library/time.po OutdatedShow resolvedHide resolved

library/time.po

		"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

eamanuAug 3, 2022

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 comoepoch, ya que así es como se lo conoce a ese instante de tiempo específico.

Copy link

ContributorAuthor

gio8tisuAug 4, 2022

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.

Añado preposicion en frase de funciones C

2b2798b

Copy link

ContributorAuthor

gio8tisu commentedAug 4, 2022

Hola@gio8tisu wave Sigue faltando que ejecutespowrap sobre el archivo que modificaste

@cmaureirpowrap en local no me aplica ningún cambio.

Copy link

Collaborator

cmaureir commentedAug 4, 2022

Hola@gio8tisu wave Sigue faltando que ejecutespowrap sobre el archivo que modificaste
@cmaureirpowrap en local no me aplica ningún cambio.

Estás en windows?

Copy link

ContributorAuthor

gio8tisu commentedAug 4, 2022

Hola@gio8tisu wave Sigue faltando que ejecutespowrap sobre el archivo que modificaste
@cmaureirpowrap en local no me aplica ningún cambio.
Estás en windows?

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

Copy link

ContributorAuthor

gio8tisu commentedAug 4, 2022

Curiosamente si ejecuto powrap sobre todos los ficheros (conmake wrap) me cambia muchos ficheros, pero nolibrary/time.po.

Copy link

Collaborator

rtobar commentedAug 4, 2022

Curiosamente si ejecuto powrap sobre todos los ficheros (conmake wrap) me cambia muchos ficheros, pero nolibrary/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 usamsgcat (parte del paquetegettext de GNU) por debajo, el cual tiene una opción para setear el ancho que tienen que tener los mensajes, pero quepowrap nunca setea. Leyendo el código demsgcat, 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.

Corrige wrapping manualmente

5ad7ef2

Copy link

ContributorAuthor

gio8tisu commentedAug 4, 2022•
edited
Loading

He logrado reproducirlo usandoact. 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.