Skip to content

Cambiando 'devolver' por 'retornar' y variaciones #531

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 3 commits into from
Jul 21, 2020

Conversation

cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Jul 6, 2020

No description provided.

@clacri
Copy link
Contributor

clacri commented Jul 8, 2020

dios, esto modifica de verdad 53 ficheros? es un cambio sobre todas las traducciones?

@cmaureir
Copy link
Collaborator Author

cmaureir commented Jul 8, 2020

dios, esto modifica de verdad 53 ficheros? es un cambio sobre todas las traducciones?

si, toca todos esos ficheros que ya teníamos merge, para estandarizar el tema.
Si no te parece, lo podemos tirar, pero igual considera que toca archivos pero no son tantos cambios por archivo hehe

Me lo puedo autorevisar también, ya que sé que es bastante trabajo.

@clacri
Copy link
Contributor

clacri commented Jul 8, 2020

ahaha no bueno si es que no me lo terminaba de creer, no pasa nada, está bien que alguien más le eche un vistazo por aquello de que a veces otros ojos ven lo que uno no.... gracias :)

@@ -1351,7 +1351,7 @@ msgstr "Todos los tipos son compatibles con :data:`Any`."
msgid ":data:`Any` is compatible with every type."
msgstr ":data:`Any` es compatible con todos los tipos."

# se añade valor para matizar que la funcion retorna (devuelve el control) pero no de manera normal. En el ejemplo lanza una excepción.
# se añade valor para matizar que la funcion retorna (retorna el control) pero no de manera normal. En el ejemplo lanza una excepción.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

es un comentario, pero no afecta el significado del mensaje :)

Copy link
Collaborator Author

@cmaureir cmaureir left a comment

Choose a reason for hiding this comment

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

Acabo de darle una segunda pasada a esto y no veo errores en el significado. De todas manera espero que alguien pueda darle un vistazo general 👍

@cmaureir cmaureir requested a review from cacrespo July 13, 2020 09:16
Copy link
Contributor

@nahueespinosa nahueespinosa left a comment

Choose a reason for hiding this comment

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

Hola! Perdón, al principio me olvide apretar el botón "Start review". Se ve todo muy bien y mis únicas dudas están en el archivo c-api/unicode.po. Leyendo el texto creo que debería ir 'retorna' en lugar de 'retorne' en algunas líneas.

Co-authored-by: Nahuel Espinosa <nahue.espinosa@gmail.com>
@cmaureir
Copy link
Collaborator Author

Toda la razón @nahueespinosa :) Muchas gracias por las sugerencias!

Copy link
Contributor

@nahueespinosa nahueespinosa left a comment

Choose a reason for hiding this comment

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

Yo creo que está todo bien 😃

Copy link
Contributor

@clacri clacri left a comment

Choose a reason for hiding this comment

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

Supongo que estáis pendientes de la revisión por @cacrespo, lo mismo, os lo dejo en approve.

@cmaureir
Copy link
Collaborator Author

Supongo que estáis pendientes de la revisión por @cacrespo, lo mismo, os lo dejo en approve.

Se lo pedí para tener otra opinión, pero ya tenemos la revisión de @nahueespinosa. Te parece @clacri si lo mergeamos?

@clacri clacri merged commit 5b85eac into python:3.8 Jul 21, 2020
@cmaureir cmaureir deleted the viva-el-retorno branch July 21, 2020 20:53
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