Skip to content

Migrar CI de TravisCI a Github Actions #1158

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

Merged
merged 8 commits into from
Dec 18, 2020
Merged

Conversation

mondeja
Copy link
Contributor

@mondeja mondeja commented Nov 28, 2020

Fixes #1157

Puedes ver esta tubería ejecutada aquí.

Si finalmente se elige otro proveedor, me parece bien, pero si se elige Github Actions aquí está la tubería para la integración contínua.

He dejado la configuración tal como estaba en TravisCI, excepto que, en lugar de usar make setup para clonar CPython, he usado los comandos de sincronización del submódulo por separado para evitar ejecutar el venv del Makefile que genera un entorno virtual instalando las dependencias dentro, ya que luego no se usa, lo que aumenta el tiempo de instalación de dependencias.

Copy link
Collaborator

@cmaureir cmaureir left a comment

Choose a reason for hiding this comment

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

A mi me parece bien el cambio, igual me gustaría saber que opina la gente que configuró travis cc @raulcd @humitos

- name: Pospell
run: |
awk 1 dict dictionaries/*.txt > dict.txt
pospell -p dict.txt -l es_AR -l es_ES **/*.po
Copy link
Collaborator

Choose a reason for hiding this comment

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

A ver si actualizo #960 y le hago merge si alguien con permisos me lo aprueba y nos evitamos los dos diccionarios.

@cmaureir
Copy link
Collaborator

cmaureir commented Dec 14, 2020

@mondeja como quitaste el travis ahora se queda pegado el check, pero recuerdo que antes pasaba así que de seguro todo está bien.
¿Te parece hacer el merge? ¿te funcionó en tu rama despues de los ultimos commits en este PR? (si tienes algún enlace a algún build de hace poco)

@mondeja
Copy link
Contributor Author

mondeja commented Dec 14, 2020

¿te funcionó en tu rama despues de los ultimos commits en este PR? (si tienes algún enlace a algún build de hace poco)

Subo un commit para poder lanzarlo manualmente, te dejo el link de mi rama y vuelvo a la configuración actual.

@mondeja
Copy link
Contributor Author

mondeja commented Dec 14, 2020

Aquí puedes verlo ejecutado para el HEAD actual en 3.8.

@cmaureir
Copy link
Collaborator

Pues yo haría el cambio lo antes posible, ya que mientras más esperemos...
@raulcd @humitos @gilgamezh @clacri ¿le damos?

@clacri
Copy link
Contributor

clacri commented Dec 16, 2020

A mi me parece bien

@cmaureir cmaureir merged commit 04d39c1 into python:3.8 Dec 18, 2020
@cmaureir
Copy link
Collaborator

Muchas gracias @mondeja 🎉
Podrías echarle un ojo a ver si algo no va bien? de seguro mucha gente va a tener que actualizar sus PRs pero es algo que teniamos que hacer en algún momento y ahora que está todo más lento por fin de año, creo que nos iría bien.

@cmaureir
Copy link
Collaborator

cmaureir commented Dec 18, 2020

@raulcd no sé si necesito permisos especiales, pero no sé quitar el check de Travis de los otros PRs, por ejemplo actualicé un PR mio: #960 y aún lo veo activado.
No puedo ver en mi perfil de travis el repo como para desactivarlo directamente.
Por ejemplo acá veo solo una lista: https://github.com/python/python-docs-es/settings/installations y no sé si puedes "quitar" alguna.

@mondeja
Copy link
Contributor Author

mondeja commented Dec 18, 2020

Podrías echarle un ojo a ver si algo no va bien? de seguro mucha gente va a tener que actualizar sus PRs pero es algo que teniamos que hacer en algún momento y ahora que está todo más lento por fin de año, creo que nos iría bien.

Claro, hecho un vistazo de vez en cuando 👍

@mondeja mondeja deleted the gh-actions branch October 1, 2021 11:16
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.

Migración de TravisCI
3 participants