X Tutup
The Wayback Machine - https://web.archive.org/web/20210117184652/https://github.com/python/python-docs-es/pull/1158
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

Migrar CI de TravisCI a Github Actions #1158

Merged
merged 8 commits into from Dec 18, 2020
Merged

Migrar CI de TravisCI a Github Actions #1158

merged 8 commits into from Dec 18, 2020

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.

@mondeja mondeja force-pushed the mondeja:gh-actions branch from b6bcfa4 to bae3b9e Nov 28, 2020
@mondeja mondeja force-pushed the mondeja:gh-actions branch from bae3b9e to ab34d31 Nov 28, 2020
Copy link
Collaborator

@cmaureir cmaureir left a comment

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

This comment has been minimized.

@cmaureir

cmaureir Nov 30, 2020
Collaborator

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

.github/workflows/main.yml Outdated Show resolved Hide resolved
@mondeja mondeja force-pushed the mondeja:gh-actions branch from 4a253cb to 24212d7 Dec 8, 2020
mondeja added 2 commits Dec 8, 2020
@cmaureir
Copy link
Collaborator

@cmaureir 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 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 mondeja force-pushed the mondeja:gh-actions branch from f281a26 to db44bea Dec 14, 2020
@mondeja
Copy link
Contributor Author

@mondeja mondeja commented Dec 14, 2020

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

@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Dec 16, 2020

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

@clacri
Copy link
Collaborator

@clacri clacri commented Dec 16, 2020

A mi me parece bien

@cmaureir cmaureir merged commit 04d39c1 into python:3.8 Dec 18, 2020
1 check passed
1 check passed
docs/readthedocs.org:python-docs-es Read the Docs build succeeded!
Details
@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Dec 18, 2020

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 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 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 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants
You can’t perform that action at this time.
X Tutup