Skip to content

Conversation

@melizeche
Copy link
Contributor

closes #774

Copy link
Collaborator

@cacrespo cacrespo left a comment

Choose a reason for hiding this comment

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

Muchas gracias @melizeche! Te dejo tan sólo una pequeña corrección.

Copy link
Contributor

@eamanu eamanu left a comment

Choose a reason for hiding this comment

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

Una vez que se agreguen los cambios sugeridos, creo que se podría
mergear tranquilamente.

Curiosidad, @melizeche te agregaste en el archivo de TRANSLATORS?

@eamanu
Copy link
Contributor

eamanu commented Feb 12, 2021

Hola @melizeche, No queda mucho para finalizar este PR :)

@melizeche melizeche requested a review from cacrespo February 12, 2021 22:12
@melizeche
Copy link
Contributor Author

@eamanu cambios hechos! Y de paso ya me agregué a TRANSLATORS 🙌

@cacrespo
Copy link
Collaborator

cacrespo commented Feb 16, 2021

Buenas @melizeche! Por alguna razón que desconozco se queda bloqueado el chequeo de readthedocs. Lo curioso es que con el resto de los pull request funciona. Te propongo que actualices tu rama con origin/3.8 y volvamos a intentar.

Aquí una ayuda de referencia.

@melizeche
Copy link
Contributor Author

@cacrespo hecho!

@cacrespo cacrespo merged commit e03cd67 into python:3.8 Feb 16, 2021
@cacrespo
Copy link
Collaborator

Felicitaciones @melizeche. Muchas gracias!!! 😄

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.

Translate 'whatsnew/index.po'

3 participants