Hola @alvarmaciel !!! Antes de continuar la revisión del contenido sería genial conseguir que el build funcionara. Te está dando errores (que puedes ver dándole a Details donde dice que Travis falló), tanto en el wrapping de las líneas (lo largas que son, lo solucionas con powrap), como en palabras que pospell, al chequear la ortografía con respecto a los diccionarios de español y nuestro diccionario de excepciones, no encuentra. Algunas son errores de verdad, otras son excepciones (términos en inglés, palabras en español que no están en el diccionario, nombres propios), que deberían incluirse en nuestro diccionario de excepciones. Las palabras son:
library/2to3.po:48:estárá
library/2to3.po:70:via
library/2to3.po:90:identación
library/2to3.po:128:casoo
library/2to3.po:140:refactorizar
library/2to3.po:140:refactorizados
library/2to3.po:140:refactorizados
library/2to3.po:160:print
library/2to3.po:179:sobreescriben
library/2to3.po:360:and
library/2to3.po:382:func
library/2to3.po:382:exec
library/2to3.po:401:call
library/2to3.po:427:trasnformaciones
library/2to3.po:427:idiomático
library/2to3.po:427:while
library/2to3.po:468:Conviert
library/2to3.po:472:isinstance
library/2to3.po:494:cmabiado
library/2to3.po:508:from
library/2to3.po:508:future
library/2to3.po:508:builtins
library/2to3.po:508:map
library/2to3.po:537:meth
library/2to3.po:549:octales
library/2to3.po:630:func
library/2to3.po:660:backtick
library/2to3.po:660:repr
library/2to3.po:688:desempaquetamiente
library/2to3.po:688:parametro
library/2to3.po:707:renombramiento
library/2to3.po:715:items
library/2to3.po:734:llamda

Una forma sencilla de revisar que el build funciona antes de hacer push es emplear pre-commit. Para hacerlo, si tienes pip, solamente necesitas

1. instalar pre-commit usando: pip install pre-commit
2. activar pre-commit en tu fork del repo de la traducción entrando en su directorio raiz y haciendo: pre-commit install

Una vez hayas seguido estos pasos, antes de hacer un commit hará esos checks y no te dejará continuar hasta que hayas solucionado los problemas. Concretamente, hace 3 cosas:

1. chequea el wrapping (el tamaño de las lineas basicamente) con powrap
2. revisa con pospell la ortografía comparando contra diccionarios de español y nuestro propio diccionario
3. reordena el fichero dict que tenemos para guardar las excepciones que queremos que pasen el pospell

Si quieres, intenta instalar pre-commit, y si necesitas ayuda porque algo no funciona, nos comentas por aqui o por telegram y vemos de solucionarlo. Un saludo y mil gracias por tu tiempo y esfuerzo!!!

Hola, Ya me pongo a esto

hola @alvarmaciel fijate que seguis teniendo algunos errores en el build de travis. Avisanos cuando este listo para hacer review o si necesitas ayuda

hola @alvarmaciel fijate que seguis teniendo algunos errores en el build de travis. Avisanos cuando este listo para hacer review o si necesitas ayuda

el último commit pasa pospell y powrap desde pre-commit

pregunta, le di resolve desde acá a lo sugerido, para actualizar mi rama es git pull upstream library/2to3
no?

y el problema no es el archivo, soy yo. prometo mejorar para el próximo 😄

diréctamente git pull debería funcionar; estos cambios no están en upstream, sino que en tu fork por el momento.

Todavía queda hacer los change request que @gilgamezh marcó (ya los hice desde los comit pero no del chage request mismo)

Ahhhh! Ya entendí lo que pasó 😄

Le diste "Resolve". Eso lo que hace es marcar esa conversación como resuelta, pero no hace efectivo el cambio que te propusieron.

Para hacer efectivo el cambio, tenés que "Add suggestion to batch" en cada una de las sugerencias y luego "Commit suggestions" en la última.

Entonces, por ahora, las sugerencias de @cmaureir y @gilgamezh todavía no están incluídas en el código en sí sino solo marcadas como "resueltas".

Me parece que yo no tengo esa opción, será que eso le aparece a quienes habilitan el merge?

@alvarmaciel deberías tener algo así:

Es probable que tengas que marcar todas las sugerencias que te hicieron como "Unresolve" para que te aparezcan.

@humitos no me aparece esa opción, mira

Lo que si pude es resolver lo conflictos del merge en el dic
creo....

@alvarmaciel justo ese 😄 comentario que mostrás en el screenshot no tiene una "sugerencia de cambio" en el código, sino simplemente un comentario.

@humitos creo que ya etá todo bien con el archivo, ahora travis no pasa por un problema de linea duplicada en dict. pero no logro ubicar esa linea

@humitos ya las encontre las estoy limpiando

eh!!!!! ya pasaro los 2 check @humitos

harmoso, primer PR aprobado en mi vida. Ahora borro el branch de mi fork, hago git pull upstream 3.8 y merge 3,8 no?