Skip to content

'Traducción archivo library/dataclasses'#481

Merged
cmaureir merged 28 commits into
python:3.8from
ignaciodopazo:traduccion-dataclasses
Aug 10, 2020
Merged

'Traducción archivo library/dataclasses'#481
cmaureir merged 28 commits into
python:3.8from
ignaciodopazo:traduccion-dataclasses

Conversation

@ignaciodopazo

@ignaciodopazo ignaciodopazo commented Jun 28, 2020

Copy link
Copy Markdown
Contributor

Closes #481

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

Copy link
Copy Markdown
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 ;)

@cmaureir cmaureir left a comment

Copy link
Copy Markdown
Collaborator

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

Comment thread library/dataclasses.po Outdated
Comment thread library/dataclasses.po Outdated
Comment thread library/dataclasses.po Outdated
Comment thread library/dataclasses.po Outdated
Comment thread library/dataclasses.po Outdated
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
Copy Markdown
Contributor Author

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

@cmaureir

Copy link
Copy Markdown
Collaborator

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

@cmaureir cmaureir left a comment

Copy link
Copy Markdown
Collaborator

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.

Comment thread library/dataclasses.po Outdated
Comment thread library/dataclasses.po Outdated
Comment thread library/dataclasses.po Outdated
Comment thread library/dataclasses.po Outdated
Comment thread library/dataclasses.po Outdated
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

@cmaureir cmaureir left a comment

Copy link
Copy Markdown
Collaborator

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?

Comment thread library/dataclasses.po Outdated
Comment thread library/dataclasses.po Outdated
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
Comment thread library/dataclasses.po Outdated
"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
Copy Markdown
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
Copy Markdown
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...

Comment thread library/dataclasses.po Outdated
Comment thread library/dataclasses.po Outdated
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
Copy Markdown
Contributor Author

En breve arreglo eso y subo algunos avances.

@cmaureir

cmaureir commented Jul 3, 2020

Copy link
Copy Markdown
Collaborator

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

@humitos humitos left a comment

Copy link
Copy Markdown
Collaborator

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

@humitos humitos left a comment

Copy link
Copy Markdown
Collaborator

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!

Comment thread library/dataclasses.po Outdated
Comment thread library/dataclasses.po Outdated
@humitos

humitos commented Jul 3, 2020

Copy link
Copy Markdown
Collaborator

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

@cmaureir

cmaureir commented Jul 9, 2020

Copy link
Copy Markdown
Collaborator

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

reopening para tener otro build

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

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