Skip to content

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

Merged
clacri merged 3 commits into
python:3.8from
cmaureir:viva-el-retorno
Jul 21, 2020
Merged

Cambiando 'devolver' por 'retornar' y variaciones#531
clacri merged 3 commits into
python:3.8from
cmaureir:viva-el-retorno

Conversation

@cmaureir

@cmaureir cmaureir commented Jul 6, 2020

Copy link
Copy Markdown
Collaborator

No description provided.

@clacri

clacri commented Jul 8, 2020

Copy link
Copy Markdown
Contributor

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

@cmaureir

cmaureir commented Jul 8, 2020

Copy link
Copy Markdown
Collaborator Author

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

clacri commented Jul 8, 2020

Copy link
Copy Markdown
Contributor

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 :)

Comment thread library/typing.po
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
Copy Markdown
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 :)

@cmaureir cmaureir left a comment

Copy link
Copy Markdown
Collaborator Author

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
Comment thread c-api/typeobj.po Outdated
Comment thread c-api/unicode.po Outdated

@nahueespinosa nahueespinosa left a comment

Copy link
Copy Markdown
Contributor

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.

Comment thread c-api/unicode.po Outdated
Comment thread c-api/unicode.po Outdated
Comment thread c-api/unicode.po Outdated
Comment thread c-api/unicode.po Outdated
Comment thread c-api/unicode.po Outdated
Comment thread c-api/unicode.po Outdated
Co-authored-by: Nahuel Espinosa <nahue.espinosa@gmail.com>
@cmaureir

Copy link
Copy Markdown
Collaborator Author

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

@nahueespinosa nahueespinosa left a comment

Copy link
Copy Markdown
Contributor

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 😃

@clacri clacri left a comment

Copy link
Copy Markdown
Contributor

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
Copy Markdown
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