-
Notifications
You must be signed in to change notification settings - Fork 396
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
Conversation
'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.
Le da mas consistencia y exactitud (véase https://es.wikipedia.org/wiki/Segundo_intercalar).
* "mucho en el futuro" por "muy adelante en el futuro" * Varios articulos * Palabras sin traducir
La palabra correcta es monótono (ver https://es.wikipedia.org/wiki/Funci%C3%B3n_mon%C3%B3tona).
Creo que es mas correcto (ver https://es.wikipedia.org/wiki/Planificador).
Buenísimo @gio8tisu 🎉 Muchas gracias! a ver si el CI pasa y lo mergeamos ;) |
There was a problem hiding this 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.
Aparte del comentario de Rodrigo, falta pasar un |
Hola, muchas gracias por los comentarios. He aplicado los cambios reescribiendo la historia de commits. |
Hola @gio8tisu 👋 |
"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 " |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
Curiosamente si ejecuto powrap sobre todos los ficheros (con |
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, Voy a ver si puedo entender por qué se produce la diferencia para poder solucionar el tema de fondo. |
He logrado reproducirlo usando act. Añadiendo el flag |
Muchas gracias @gio8tisu 🎉 🎉 |
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.