Skip to content

Migrar CI de TravisCI a Github Actions#1158

Merged
cmaureir merged 8 commits into
python:3.8from
mondeja:gh-actions
Dec 18, 2020
Merged

Migrar CI de TravisCI a Github Actions#1158
cmaureir merged 8 commits into
python:3.8from
mondeja:gh-actions

Conversation

@mondeja

@mondeja mondeja commented Nov 28, 2020

Copy link
Copy Markdown
Contributor

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.

@cmaureir cmaureir left a comment

Copy link
Copy Markdown
Collaborator

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
Copy Markdown
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.

Comment thread .github/workflows/main.yml Outdated
@cmaureir

cmaureir commented Dec 14, 2020

Copy link
Copy Markdown
Collaborator

@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

mondeja commented Dec 14, 2020

Copy link
Copy Markdown
Contributor Author

¿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

mondeja commented Dec 14, 2020

Copy link
Copy Markdown
Contributor Author

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

@cmaureir

Copy link
Copy Markdown
Collaborator

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

@clacri

clacri commented Dec 16, 2020

Copy link
Copy Markdown
Contributor

A mi me parece bien

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

Copy link
Copy Markdown
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

cmaureir commented Dec 18, 2020

Copy link
Copy Markdown
Collaborator

@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

mondeja commented Dec 18, 2020

Copy link
Copy Markdown
Contributor Author

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