API_Hacienda icon indicating copy to clipboard operation
API_Hacienda copied to clipboard

Fix docker compose

Open eddyvalverde opened this issue 3 years ago • 3 comments

  1. Cambio de End of File de CRLF a LF a todos los archivos porque el servicio docker de php no funcionaba (standard_init_linux.go:190: exec user process caused "no such file or directory")
  2. Agregar Adminer como servicio al archivo de docker compose para accesar a la base de datos desde el puerto 8081
  3. Agregar example.env que son variable de entorno utilizadas por la app

eddyvalverde avatar Jun 02 '22 23:06 eddyvalverde

Hola @eddyvalverde Pura vida por el PR. Consulta, a qué se debe los cambios en estos archivos?

api/contrib/baseModule/module.php
api/contrib/signXML/Firmadohaciendacr.php
api/core/boot.php
api/core/modules.php
api/core/tools.php
api/modules/crypto/crypto.php
api/modules/crypto/module.php

?? @eddyvalverde

Consulta, a qué se debe los cambios en estos archivos?

Solo son cambios de retorno de carro CRLF a LF. Si en GitHub visualizas omitiendo cambios de espacio en blanco también se omiten estas diferencias, por lo que es fácil comprobar que no hay otras modificaciones. Esto lo menciona en el punto 1 de la descripción. Lo que no queda tan claro es por qué esto es necesario para que Docker no dé errores. De todos modos el cambio es relativamente inocuo (el "git blame" puede requerir ir un commit más atrás para analizar las líneas).

Ahora que el Bloc de notas de Windows 10 en adelante reconoce los saltos de línea tipo UNIX, podría agregarse un .editorconfig (ver https://editorconfig.org/ para más información) y aplicar la regla masivamente, Sin olvidar un fichero .gitattributes con las líneas * text=auto y *.php text, por ejemplo, para que los saltos de línea al agregarlos a git se manejen adecuadamente de forma automática en cada sistema operativo de la manera nativa.

fdelapena avatar Jun 12 '22 03:06 fdelapena