open-source-uc / auto-ramos

Tomador de ramos automatico UC

GNU General Public License v3.0

13 stars 4 forks source link

Nueva versión de la sugerencia #3

Closed eanorambuena closed 3 years ago

eanorambuena commented 3 years ago

1 #2

Dyotson commented 3 years ago

¡Probare tu branch a ver que tal en el reajuste!

agucova commented 3 years ago

@Dyotson Pregunta, van a implementar reviews como requisito? Si es así, puedo activar la configuración que usamos en ramos y salas UC.

agucova commented 3 years ago

Y @eanorambuena, no entiendo, esto sustituye #2? (Si es así, hay que cerrarlo)

También, como tip, intenta describir brevemente cual es el propósito y contexto a cada PR, y si es posible, preguntas que tengas o cosas que quieres que alguien revise. Asi es más fácil que los reviewers entiendan más rápidamente qué hiciste y si es que necesita cambios.

eanorambuena commented 3 years ago

@agucova Sí, sigue el hilo #1 #2. Por eso los cité. No me dejaba ponerlo en #2, no comprendo por qué, así que tuve que hacer otro. No sabía que los podía cerrar yo. Gracias por el tip. PD: la línea de tiempo está en #1, salen los Pull Requests y commits asociados.

Dyotson commented 3 years ago

Debido a cambio en version de auto-ramos, este codigo queda deprecado. Este codigo estaba muy bueno, pero lamentablemente es incompatible con la nueva version, seria genial si pudieras reeimplementarlo.

eanorambuena commented 3 years ago

Eso veo, me alegra el progreso que están teniendo. Éxito, cuando pueda veo si lo puedo reimplementar.