Rescatando turpial de entre los muertos

8 views
Skip to first unread message

Abraham Toriz

unread,
Feb 12, 2022, 2:49:39 PM2/12/22
to Turpial-Dev
Que tal, escribiendo por aquí a petición de @satanás y para tener registro de lo sucedido.

Hasta ahora lo que tengo es una versión sintácticamente compatible con python 3, pero los tests están rotos pues una dependencia importante, oauth, está desactualizada. Ante esto mi propuesa es utilizar requests-oauthlib que añade soporte de oauth a la ya utilizada biblioteca requests. Este paquete tiene buen soporte hasta el día de hoy.

El código alterado por esto afortunadamente se reduce a no más de dos centenas de líneas de código altamente comentadas en libturpial/lib/http.py. Mi pregunta en este punto es cuál es el flujo de desarrollo que se me recomienda seguir. ¿Cómo pruebo lo que hago?

Saludos y muchas gracias

Wil Alvarez

unread,
Feb 12, 2022, 10:10:10 PM2/12/22
to turpi...@googlegroups.com
Hola Abraham,

De nuevo, gracias por iniciar el hilo de discusión por acá.

Estuve revisando requests-oauthlib[1] y se ve como una buena candidata para reemplazar la vieja oauth. No he tocado el código en mucho tiempo como para saber si es un drop-in replacement ni qué tanto esfuerzo es necesario para migrar pero si dices que son unas doscientas líneas de código suena como algo plausible. Para el flujo de desarrollo, creo que pull requests con cambios pequeños y frecuentes son la mejor opción; tienen scope limitado y son más fáciles de testear y revisar. Voy a hacer una rama llamada python3 para que todos los pull requests parciales vayan contra esa rama hasta que tengamos una versión funcional. Una vez confirmado que todo anda bien, mergeamos contra development y hacemos un release oficial :)

Para el tema de probar tus cambios, los test unitarios deberían cubrir en cierta medida el comportamiento pero hasta donde recuerdo no hay tests de integración, así que la funcionalidad toca probarla a mano. Lo que hacía en su momento era crear una instancia de libturpial en la consola de python, autenticarme y hacer tweets de prueba (esos pasos están en el readme[2]). Con eso y los tests unitarios creo que sería más que suficiente por el momento. Igual, si ves que hace falta algo de cobertura y quieres hacer backfill de tests unitarios, bienvenidos sean, pero diría que enfoquemos los esfuerzos en migrar primero.

Lo otro que me gustaría saber es en qué estado está el PR que mandaste en 2020 [3]. Si crees que esos cambios todavía son válidos entonces empiezo por hacer code review de eso y mergearlo a la rama python3 de una.

Saludos.


--
You received this message because you are subscribed to the Google Groups "Turpial-Dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to turpial-dev...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/turpial-dev/c96114d7-2927-46de-97e2-1f2897c13b32n%40googlegroups.com.


--
Wil A. Alvarez
Linux Counter #415026
Debian Counter #259

Wil Alvarez

unread,
Feb 13, 2022, 6:05:36 PM2/13/22
to turpi...@googlegroups.com
¡Tenemos el primer cambio mergeado en la rama python3[1]! 🙌

Gracias Abraham por tomar la iniciativa. Si hay algo más que quieras conversar o algo en lo que te pueda orientar que te esté bloqueando el progreso, avísame y con gusto lo hago.

José Subero

unread,
Feb 15, 2022, 5:46:33 PM2/15/22
to turpi...@googlegroups.com
Hey, si van a retomar esto, avisen, para ponerme al dia y al menos
darle testing al pajarito... :-D

El sáb, 12 feb 2022 a la(s) 22:10, Wil Alvarez
(wil.al...@gmail.com) escribió:
> To view this discussion on the web visit https://groups.google.com/d/msgid/turpial-dev/CAMBtWNUzPp6LTCs80dXEF%3DyXQCFBFmodzjpTc8iqy37g%2B79-LA%40mail.gmail.com.



--

Muchas veces dilemas en las tecnologías se resuelven sin tecnología,
solo se necesita sentido común y un buen uso de Google.

Quien suscribe:
José Leonel Subero Gamarra
@arawako
Linux Counter #438135
Huella de clave = 0752 0C94 0DE4 77F5 078B FE3A F671 B8EA 0F5E 49D7

http://www.arawako.com
http://www.clug.org.ve
http://www.mundoaccesible.org.ve
http://www.gentoove.org
http://www.entrehackers.com
http://www.turpial.org.ve
Reply all
Reply to author
Forward
0 new messages