Skip to content

'Traducción archivo library/dataclasses' #481

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 28 commits into from
Aug 10, 2020
Merged

'Traducción archivo library/dataclasses' #481

merged 28 commits into from
Aug 10, 2020

Conversation

ignaciodopazo
Copy link
Contributor

@ignaciodopazo ignaciodopazo commented Jun 28, 2020

Closes #481

@ignaciodopazo ignaciodopazo changed the title 'Traducción archivo dataclasses' 'Traducción archivo library/dataclasses' Jun 28, 2020
@cmaureir
Copy link
Collaborator

Gracias @ignaciodopazo por la traducción! :D si miras la salida del build, puedes ver que hay unos problemillas:

$ pospell -p dict -l es_AR -l es_ES **/*.po

<rst-doc>:1: (WARNING/2) Inline interpreted text or phrase reference start-string without end-string.

<rst-doc>:1: (WARNING/2) Inline interpreted text or phrase reference start-string without end-string. while parsing: Si :func:`dataclass`es llamada sin parámetros, actúa con los valores por defecto documentados aquí. Los siguientes tres usos de :func:`dataclass`son equivalentes
library/dataclasses.po:26:classes
library/dataclasses.po:109:metodos
library/dataclasses.po:122:func
library/dataclasses.po:122:dataclass
library/dataclasses.po:122:func
library/dataclasses.po:122:dataclass
library/dataclasses.po:149:string

De seguro la mayoría son por temas de sintaxis de los roles de sphinx, a ver si le echas un vistazo ;)

Copy link
Collaborator

@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.

A ver si con estas sugerencias se van los problemas del build :)

ignaciodopazo and others added 5 commits June 28, 2020 20:06
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@ignaciodopazo
Copy link
Contributor Author

Gracias por las sugerencias @cmaureir , ahí voy a contentar al build!

@cmaureir
Copy link
Collaborator

Genial! ahí está pasando, ahora le echaré un vistazo al archivo :)

Copy link
Collaborator

@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.

Ya casi estamos, solo dejé un par de comentarios menores.

ignaciodopazo and others added 5 commits June 29, 2020 13:08
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@cmaureir
Copy link
Collaborator

vas a tener que pasarle un powrap a tu archivo, que de seguro con mis cambios algunas líneas quedaron muy largas.

@ignaciodopazo
Copy link
Contributor Author

Hecho lo del powrap @cmaureir! Hice algunos avances y veo que el build genera problemas, ¿puede ser porque el diccionario de pospell en español no acepta ciertas palabras? Por ejemplo algunas derivadas de hash, que traducirlas al español me genera dudas. Por ahora siempre he visto que esa palabra en particular aparece sin traducir en la literatura en español, pero no he leído todo así que puede fallar :)

@cmaureir
Copy link
Collaborator

Sí, es justamente porque pospell no reconoce algunas.
Yo creo que hashing la podrías dejar entre asteríscos, asi pasa el build (también con unhashable)
pero voy a mirar las líneas que si tienes espacios extras, por ejemplo frozen an vez de frozen, no se reconocerá. Dejame darle un vistazo a ver si encuentro algo, si no, las puedes agregar con asteríscos.

 pospell -p dict -l es_AR -l es_ES **/*.po
library/dataclasses.po:280:frozen
library/dataclasses.po:280:unhashable
library/dataclasses.po:280:meth
library/dataclasses.po:280:hashing
library/dataclasses.po:306:frozen

Copy link
Collaborator

@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.

yo creo que esto arreglará un par de errores, y puedes agregar "hash" y "hasheable" al diccionario. Creo que tú @humitos revisaste algo similar en otro PR, no?

ignaciodopazo and others added 2 commits July 1, 2020 16:12
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@cmaureir cmaureir self-requested a review July 1, 2020 19:20
@@ -199,6 +293,14 @@ msgid ""
"superclass will be used (if the superclass is :class:`object`, this means it "
"will fall back to id-based hashing)."
msgstr ""
"Si a ``eq`` y a ``frozen `` son asignadas true, :func:`dataclass` genera "
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"Si a ``eq`` y a ``frozen `` son asignadas true, :func:`dataclass` genera "
"Si ``eq`` y ``frozen`` son true, :func:`dataclass` genera "

Copy link
Collaborator

Choose a reason for hiding this comment

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

Si eq y frozen son verdaderos...

Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
ignaciodopazo and others added 5 commits July 1, 2020 16:25
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@ignaciodopazo
Copy link
Contributor Author

En breve arreglo eso y subo algunos avances.

@cmaureir
Copy link
Collaborator

cmaureir commented Jul 3, 2020

Por cierto, como aún sigues trabajando con la traducción, quizás puedes hacer un merge de la branch 3.8, asi te traes los cambios que han pasado en el dict :)

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.

Marco como Request Changes porque me gustaría mirarla antes de hacer merge de esta

humitos
humitos previously requested changes Jul 3, 2020
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.

Ahí revisé lo que pensé que estaba mal y, estaba bien 😅

Igualmente, encontré otros pequeños detalles para cambiar antes de hacer merge. ¡Gracias por su trabajo!

@humitos
Copy link
Collaborator

humitos commented Jul 3, 2020

Marqué algunos comentarios como "Unresolved" así los revisamos. Luego de eso, estamos para hacer merge!

@cmaureir
Copy link
Collaborator

cmaureir commented Jul 9, 2020

@ignaciodopazo los errores del build ya fueron solucionados. Te recomiendo que hagas un merge de 3.8 en tu rama local y vuelvas a hacer un push, de seguro todo sale bien :)

@cmaureir
Copy link
Collaborator

A mi me parece bien, veamos que dice @humitos ;)

Por cierto, hiciste un commit sin mensaje en el último, igual intentemos poner cosas como "Merge de 3.8", "powrap al archivo xxx/xxxx", o "Agregando palabra faltante" que si no, después nos perdemos de que hacia cada commit 👍

@ignaciodopazo
Copy link
Contributor Author

Mala mía! sería 'Merge 3.8' ese commit sin comentar.

@cmaureir cmaureir dismissed humitos’s stale review July 25, 2020 12:46

+1 desde discord

@cmaureir cmaureir closed this Jul 25, 2020
@cmaureir cmaureir reopened this Jul 25, 2020
@cmaureir
Copy link
Collaborator

reopening para tener otro build

@cmaureir cmaureir merged commit 98a57ac into python:3.8 Aug 10, 2020
@cmaureir
Copy link
Collaborator

Se me había pasado hacer el merge de esto @ignaciodopazo muchas muchas gracias 🎉 🎉

@ignaciodopazo ignaciodopazo deleted the traduccion-dataclasses branch October 17, 2020 21:09
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