mozky
commented
7 years ago @luisaguilar2910 @sainoba @LuisEvilCo @Sler69 que opinan?
Open mozky opened 7 years ago
mozky
commented
7 years ago @luisaguilar2910 @sainoba @LuisEvilCo @Sler69 que opinan?
sainoba
commented
7 years ago Yo sólo diría que hay que tener cuidado con lo que soltamos de métricas. No nos vaya a pasar como a los diseñadores que a cada rato les reclaman de no andar cumpliendo sus cuotas por andar jugando al chido en su competencia/trampa.
mozky
commented
7 years ago @sainoba si totalmente de acuerdo, pero no creo que el proyecto vaya tanto en ese sentido, sino que Ro quiere algo para saber en que proyecto está trabajando cada quien y como vamos avanzando, pero a lo que entiendo si tiene claro de que en desarrollo no se puede implementar algo de X numero de PRs a la semana
Contexto
La forma en la que obtenemos los PRs actualmente es haciendo primero un request a github/edlio y luego por cada repo le pedimos sus PRs, lo cual genera un bueeeeen de requests, y mas si queremos que se esté actualizando la info en tiempo real, al menos por ahorita usando REST
Problema
Usando personal tokens tenemos un limite de 5000 requests al día, obviamente nos las acabamos si usamos varias personas la aplicación o si esta activado el refresh...
Ideas de posibles soluciones
Por cada request Github regresa un header llamado "ETag" el cual sirve como cache si se envía en las siguientes peticiones, si la info no ha cambiado, github regresa un 304 y no se gastan nuestras requests disponibles.. Ya hice la parte de guardar estos ETags por cada repo, pero el problema es que usamos una librearia (github-api) la cual no permite custom headers (bullshit).. Entonces para poder usar esta solución debemos: o cambiar de librería o crear las peticiones a github desde 0 (hueva)
Usar basic authentication, lo cual sería pedirle al usuario su username y password de github al abrir la aplicación y de esta forma hacemos las peticiones a su nombre, quitando el personal token y asi ya no tendriamos problemas de requests limit (creo)
Hacer dev_stats 2.0 y hacerlo una GitHub Application chida con OAuth, webhooks y toda la madre para asi github nos notifique de nuevos PRs y tendríamos un request limit mucho mas alto (mucho trabajo por ahorita en mi opinión)