Skip to content

Traducido c-api/datetime #292

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 5 commits into from
Jun 7, 2020
Merged

Conversation

cmaureir
Copy link
Collaborator

No description provided.

@cmaureir cmaureir force-pushed the traduccion-datetime branch from 1ab7833 to 1d58182 Compare May 21, 2020 14:19
@cmaureir cmaureir force-pushed the traduccion-datetime branch from 1d58182 to d9eb8b5 Compare May 23, 2020 13:52
Copy link
Collaborator

@humitos humitos left a comment

Choose a reason for hiding this comment

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

Está bien. Dejé algunos comentarios y sugerencias.

@@ -34,114 +36,153 @@ msgid ""
"structure into a static variable, :c:data:`PyDateTimeAPI`, that is used by "
"the following macros."
msgstr ""
"El módulo :mod:`datetime` proporciona varios objetos de fecha y hora. Antes "
Copy link
Collaborator

Choose a reason for hiding this comment

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

Acá estás cambiando de voz pasiva a voz activa. A mí particularmente me gusta tu cambio ya que queda mucho más lejible, aunque quizás no sea 100% correcto. Por mí 👍

@raulcd
Copy link
Collaborator

raulcd commented Jun 6, 2020

@cmaureir Qué te parecen los comentarios de @humitos ? Los aplicamos y mergeamos?

Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
@cmaureir
Copy link
Collaborator Author

cmaureir commented Jun 7, 2020

Gracias por los comentarios, ya los acepté y agregué veamos si es que tengo que agregar alguna palabra al dict y ya estariamos

@cmaureir
Copy link
Collaborator Author

cmaureir commented Jun 7, 2020

@raulcd @humitos creo que ya estamos

@humitos humitos merged commit 2d7962f into python:3.8 Jun 7, 2020
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.

3 participants