Skip to content

Traducido archivo library/doctest #1144

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 65 commits into from
Apr 1, 2021
Merged

Conversation

aroquega
Copy link
Collaborator

Closes #348

@Bgeninatti
Copy link

Hola @aroquega
Muchas gracias por tu traducción

El build de Travis está fallando por algunas palabras que no tenes en tu diccionario:

library/doctest.po:99:doctests
library/doctest.po:118:doctest
library/doctest.po:174:doctest
library/doctest.po:207:doctest
library/doctest.po:268:doctest
library/doctest.po:268:estandar
library/doctest.po:299:doctest
library/doctest.po:356:doctest
The command "pospell -p dict.txt -l es_AR -l es_ES **/*.po" exited with 255.

Seguramente sea porque las palabras en inglés tienen que ir en cursiva (rodeada por *), como por ejemplo doctest que aparece varias veces.

Si resolvés eso ya va a pasar el build y vamos a poder empezar la revisión 😀

@aroquega
Copy link
Collaborator Author

¿@Bgeninatti estar de acuerdo en poner doctest en el diccionario? en vez de ponerlo entre '*'

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.

un mínimo cambio para nada importante.

@Bgeninatti
Copy link

¿@Bgeninatti estar de acuerdo en poner doctest en el diccionario? en vez de ponerlo entre '*'

Si, estoy de acuerdo en poner doctest en el diccionario ya que no es una palabra en inglés sino que es un término específico de Python.

@eamanu
Copy link
Contributor

eamanu commented Nov 21, 2020

@aroquega Estoy viendo que el pospel está mostrando un error de un cambio introducido en ee86f03

@aroquega
Copy link
Collaborator Author

@eamanu no creo que el error haya sido causado por cambio de ee86f03.

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.

Lo veo bien, cuando creaas que está listo, le sacas el Draft \o/

@eamanu
Copy link
Contributor

eamanu commented Nov 21, 2020

@aroquega no no ahora vi que no, pero está tirando un mensaje raro que estoy mirano

<rst-doc>:1: (ERROR/3) RFC number must be a number greater than or equal to 1; "RFC 2045 section 6.8 <2045#section-6.8>" is invalid. while parsing: Los datos codificados tendrán líneas nuevas cada 76 caracteres según :rfc:`RFC 2045 section 6.8 <2045#section-6.8>`, que era la especificación estándar de facto base64 cuando se escribió la especificación XML-RPC.

@eamanu
Copy link
Contributor

eamanu commented Dec 3, 2020

@aroquega hola.. le falta algo más al pr

@aroquega
Copy link
Collaborator Author

aroquega commented Dec 3, 2020

Hola @eamanu sí me falta, estaba en la sección de banderas de este doc y aún no lo he subido. Este fin voy a seguir avanzando.

@eamanu
Copy link
Contributor

eamanu commented Dec 3, 2020

ah genial! :)

@cmaureir
Copy link
Collaborator

Hola @aroquega te podemos dar una mano con este PR?

@aroquega
Copy link
Collaborator Author

Hola @cmaureir, he estado ocupado. Me quedé traduciendo la parte de banderas pero están en necesita revisión porque aún no las he revisado con detalle.
Voy seguir con la traducción poquito a poquito.

@cmaureir cmaureir added the work-in-progress This PR is still work in progress label Mar 20, 2021
@cacrespo
Copy link
Collaborator

cacrespo commented Mar 20, 2021

Hola @aroquega! He impactado en el pull request unos ajustes que necesitamos hacer a partir del cambio de base 3.8 a 3.9 en la traducción.

Por otro lado, me encontré con un error en el build de la documentación a partir de esto:

If any reporting flags "
"were specified when the :class:`DocTestCase` instance was constructed, "
":mod:`doctest`'s :mod:`unittest` reporting flags are ignored."

Si traducimos ...:mod:`unittest` de :mod:`doctest`... (que tiene todo el sentido del mundo en el español) se invierte el orden de esas palabras clave y lanza un warning. Así que hice uso de mi imaginación literaria 😋 para adaptarlo:
...se ignoran las banderas de informe -pertenecientes a mod:`doctest`- de :mod:`unittest`.
¿Qué te parece? Tal vez convendría validarlo con alguien más en el grupo de Telegram.

Por lo que he visto hasta aquí, muy buen trabajo!!! 🥳
¿Hay algo que podamos hacer para ayudarte a completar la traducción?

@aroquega
Copy link
Collaborator Author

Hola @cacrespo , genial tu solución, yo me frustré un poco porque no podía resolver los warnings y errores del postpell. Por mucho rato, dedicaré un tiempo ahora para terminarlo.

@aroquega aroquega marked this pull request as ready for review March 23, 2021 23:02
@aroquega aroquega requested a review from cacrespo March 23, 2021 23:18
@cacrespo
Copy link
Collaborator

Qué bueno que lo hayas terminado @aroquega. 💪 💪 💪
Por favor, dame unos días y lo reviso.
Saludos

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.

Tremendo trabajo has realizado!!! 😄
Van algunas sugerencias y muchas gracias por el esfuerzo.

@aroquega
Copy link
Collaborator Author

Gracias @cacrespo por tus observaciones,relamente hubo partes donde estaba en modo automático. Sólo quiero conversar dos observaciones donde comenté y ya estaría.

@cacrespo
Copy link
Collaborator

@aroquega te hice la devolución de los comentarios. (Te aviso por si se te pasó de largo)
Abrazo

@aroquega aroquega requested a review from cacrespo April 1, 2021 16:29
@cacrespo cacrespo merged commit d094c84 into python:3.9 Apr 1, 2021
@cacrespo
Copy link
Collaborator

cacrespo commented Apr 1, 2021

Felicitaciones @aroquega !!!! 🥳🥳🥳🥳

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
work-in-progress This PR is still work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Translate library/doctest.po
6 participants