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

[IV-21-22] Objetivo 6 #23

Merged
merged 9 commits into from
Mar 3, 2022
Merged

[IV-21-22] Objetivo 6 #23

merged 9 commits into from
Mar 3, 2022

Conversation

arguellesm
Copy link
Owner

@arguellesm arguellesm commented Feb 8, 2022

Description

Checklist

  • ¿Se ha evitado Circle CI, porque a estas alturas ya lo ha usado todo el
    mundo?
  • ¿Se han establecido criterios a priori para elegir el sistema de
    integración continua más conveniente y se han calificado los sistemas que
    cumplan esos requisitos según esos criterios?
  • ¿Se han examinado varios sistemas de integración continua?
  • ¿Se han configurado varios sistemas de integración continua?
  • ¿Se ha configurado correctamente el Checks API en los sistemas en los que
    sea necesario para que aparezca correctamente en GitHub (y se pueda comprobar
    desde los tests)?
  • ¿Uno de los sistemas configurados permite comprobar cuales son las versiones
    del lenguaje con las que funciona correctamente nuestra aplicación?
  • ¿Se escogen de forma adecuada las versiones del lenguaje que se testean,
    tanto en el sistema de CI como en el contenedor Docker?
  • ¿Se han justificado correctamente las versiones del lenguaje que se están
    testeando y se ha comprobado que no se comprueban varias veces lo mismo (la
    misma versión en CI y en el contenedor Docker en otro sistema CI?

Con esta PR se avanza #2.

@arguellesm arguellesm added the ci label Feb 8, 2022
@arguellesm arguellesm added this to the 0.2.0 milestone Feb 8, 2022
@arguellesm arguellesm self-assigned this Feb 8, 2022
@arguellesm
Copy link
Owner Author

Best practices followed when creating actions and pipelines:

  • Used matrix to test for untested Python versions.
  • Used a container (the one we uploaded to DockerHub) to run unit tests in GitLab.
  • It only made sense to test for Python versions 3.8 and 3.9 because of dependencies. This decision is documented in CI_systems.md. Python 3.9 is already being tested inside the Docker container with a GitLab CI pipeline, which meant that the GitHub action for testing Python versions only needed to test 3.8.

@JJ
Copy link

JJ commented Feb 9, 2022

⛹ Revisores → @alexespana @Olasergiolas @paula1999

@JJ
Copy link

JJ commented Feb 9, 2022

El milestone en el que está incluido no incluye CI. La descripción del producto mínimamente viable debe incluir todos los issues/PRs que están asignados a ella.

@arguellesm arguellesm modified the milestones: 0.2.0, 0.2.1 Feb 9, 2022
@arguellesm
Copy link
Owner Author

Copy link

@JJ JJ left a comment

Choose a reason for hiding this comment

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

👍

@arguellesm arguellesm merged commit 298ed5a into objetivo-5 Mar 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants