[IV-23-24] Objetivo 5 - Githubissues

DarckMonster commented 9 months ago

[x] ¿Se han establecido criterios previos de elección de la imagen base, así como los criterios de búsqueda que han conducido a las imágenes que se están considerando?
[x] ¿Se ha tenido en cuenta que la imagen es para testear, y que por tanto tiene una función específica, tal como testear cómo va nuestra aplicación en la última versión publicada del lenguage?
[x] ¿Se ha tenido en cuenta que la versión de la imagen base sea la última?
[x] ¿Se han tenido en cuenta criterios reales para la elección, y no criterios irrelevantes como cuanto tiempo se tarda en construirla? (Se va a construir una vez, se va a usar cientos o miles de veces)
[x] ¿Se han seguido las buenas prácticas en las órdenes que construyen la imagen? ¿Especialmente las relativas a usuarios no privilegiados y demás?
[x] ¿La imagen incluye sólo y exclusivamente la infraestructura necesaria para pasar los tests? Por ejemplo, si se ha elegido la imagen "oficial" (lo que hay que hacer sólo en casos muy justificados) ¿se ha comprobado que lo que se instale sea sólo y exclusivamente lo necesario para nuestro proyecto?
[ ] ¿Se han documentado y enlazado los commits a las imágenes que se han testeado?
[x] ¿Está establecido correctamente como WORKDIR el que se va a usar en los tests que se van a lanzar?
[ ] ¿Se usa algún directorio adecuado para copiar contenidos del repositorio en vez del directorio de trabajo?
[x] ¿Se han establecido claramente los criterios de búsqueda y de aceptación de la imagen base?
[x] ¿Tienes claro cuales son las condiciones en las que tiene que actualizarse la imagen en DockerHub y has configurado el workflow de acuerdo con ello? En particular, ¿sabes qué condiciones tienen que darse para que se actualice y qué condiciones no deben darse?
[x] Si se usan GitHub actions para su publicación, ¿se han considerado cuidadosamene los criterios (ramas, paths) para que se lancen?
[x] ¿Se ha comprobado que el ENTRYPOINT del contenedor no escriba nada en el directorio que monta, ya que no tiene derecho de escritura?
[ ] ¿Se ha abierto un shell en el contenedor una vez construida la imagen y comprobado que los ficheros que se necesitan están todos incluidos, y que no hay ficheros que no se necesiten?

closes #56 closes #59

DarckMonster commented 9 months ago

07929fcbaae3061050855f8de8c07d4e6b45f9c9 commit por error :no_mouth: es referido a #55

Enaraque commented 9 months ago

En lo correspondiente a los commits, intenta cerrar todos los issues con ellos ya que hay algunos donde haces referencia a algunos de ellos pero no los cierras después, como en #56 o #59 🫡

DarckMonster commented 9 months ago

En lo correspondiente a los commits, intenta cerrar todos los issues con ellos ya que hay algunos donde haces referencia a algunos de ellos pero no los cierras después, como en #56 o #59 🫡

CIerto, 🫡, en los otros pr me ha pasado mucho, ya he aprendido jaj

Enaraque commented 9 months ago

Yo lo veo listo para revisión también 👌

DarckMonster commented 9 months ago

@JJ dos correcciones positivas

DarckMonster / PCscrap

[IV-23-24] Objetivo 5 #50