Skip to content

Traduccion whatsnew/2.7.po #1159

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 20 commits into from
Dec 20, 2020
Merged

Conversation

Juerodriguez
Copy link
Contributor

Porcentaje de avance 14%

@Juerodriguez Juerodriguez marked this pull request as ready for review December 3, 2020 14:54
@Juerodriguez
Copy link
Contributor Author

Cuando lo paso por pospell me marca el siguiente problema: :1: (WARNING/2) Inline interpreted text or phrase reference start-string without end-string. while parsing: Comparando a un :class:~collections.OrderedDictcon un diccionario normal ignora el orden de inserción y simplemente compara las llaves y los valores.

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.

Dejo algunas sugerencias

@eamanu
Copy link
Contributor

eamanu commented Dec 16, 2020

@Juerodriguez marqué algunas, pero fijate que en los párrafos o frases que terminan con :: en inglés, también deben quedar así en español.

Este es un archivo largo, así que podríamos dejarlo en un 25%? y luego en otro PR sigues con el otro 25%, y así... que te parece?

@eamanu
Copy link
Contributor

eamanu commented Dec 16, 2020

fijate además los errores del Build. Cualuiqer duda que tengas o si necesitas ayuda, avisame :)

Juerodriguez and others added 12 commits December 16, 2020 16:00
Co-authored-by: Emmanuel Arias <eamanu@yaerobi.com>
Co-authored-by: Emmanuel Arias <eamanu@yaerobi.com>
Co-authored-by: Emmanuel Arias <eamanu@yaerobi.com>
Co-authored-by: Emmanuel Arias <eamanu@yaerobi.com>
Co-authored-by: Emmanuel Arias <eamanu@yaerobi.com>
Co-authored-by: Emmanuel Arias <eamanu@yaerobi.com>
Co-authored-by: Emmanuel Arias <eamanu@yaerobi.com>
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.

Te dejo un solo cambio. Y si te parece @Juerodriguez le pedimos a los otros chicos otro review + merge, y continuas con otro porcentaje (o no, cambias a otro archivo)

@cmaureir cmaureir marked this pull request as draft December 20, 2020 21:09
@cmaureir
Copy link
Collaborator

Lo dejo como draft hasta que esté 100%

@eamanu
Copy link
Contributor

eamanu commented Dec 20, 2020

@cmaureir Había hablado con @Juerodriguez para que vamos mergeando de a 25%, porque sino se hace pesado revisar, es una file algo grande. Que te parece?

@cmaureir
Copy link
Collaborator

@eamanu ningún problema! entonces hacemos el merge ahora, no?

@eamanu
Copy link
Contributor

eamanu commented Dec 20, 2020

Si si, ahi seguimos trabajando en otro 25%

@cmaureir cmaureir marked this pull request as ready for review December 20, 2020 21:51
@cmaureir cmaureir merged commit f98f60d into python:3.8 Dec 20, 2020
@Juerodriguez Juerodriguez deleted the traduccion-whatsnew/2.7.po branch January 6, 2021 18:59
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