Skip to content
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

Feature/add GitHub actions #207

Closed
wants to merge 7 commits into from
Closed

Conversation

lecovi
Copy link
Contributor

@lecovi lecovi commented May 22, 2021

No description provided.

@lecovi lecovi closed this May 22, 2021
@lecovi lecovi reopened this May 22, 2021
@tzulberti
Copy link
Collaborator

Buenas. Porque usar Github Actions en vez de travis? Justo cree un ticket el otro dia para que todos los repos usen los mismo (PyAr/pyar_infra#81), y me interesaria saber porque cambiar uno por el otro.

Por el otro lado, creo que el PR de bumpear la version de Django debería ser la base para este PR porque figuran muchos cambios de ese PR

@facundobatista
Copy link
Member

Buenas. Porque usar Github Actions en vez de travis?

Travis entró en una fase de migración de organización pública a comercial o algo parecido.

La otra vez estuve peleando un montón con dónde están los repos de PyAr, si travis.com o travis.org, y no los podía hacer andar, el final fue feliz ahí pero fue un laburo.

Este cambio acá está originado porque Leandro metió un PR, y yo me di cuenta que Travis, UNA VEZ MÁS, no contestaba. Me pareció bien ir a GA.

Leandro después investigó, y encontró que parece que se nos acabó el crédito en Travis (?!), adjunto screenshot.

Más allá de eso, sí, en este PR está todo mezclado.

Copy link
Member

@facundobatista facundobatista left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quedó todo mezclado en este PR, hay que separar lo de GA solito... Gracias!

@lecovi
Copy link
Contributor Author

lecovi commented May 25, 2021

Si, este PR quedó mezclado porque quería ver GA y creé un branch a partir de mi PR, no me di cuenta que tendría que haberlo hecho a partir de master.
BTW, mi PR #206 está basado a partir del #203 así que podríamos jugar con ese #203.

Cierro esto para con genera más ruido.

@lecovi lecovi closed this May 25, 2021
@lecovi lecovi deleted the feature/add-github-actions branch May 25, 2021 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants