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

Fix files not being correctly unzipped #101

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rogervila
Copy link
Contributor

Hola @ssheduardo,

Este PR arregla un issue durante la instalación del paquete en entornos en los que unzip no acepta acentos en los nombres de fichero.

Captura de pantalla 2024-03-27 a las 11 30 18

Además, reduce el tamaño de la librería ignorando ficheros y carpetas durante la instalación con composer que no se usan en el paquete.

@ssheduardo
Copy link
Owner

Entiendo que este error te da cuando le das a descargar en Zip la librería?

@rogervila
Copy link
Contributor Author

Composer usa la librería unzip del sistema durante la instalación del paquete. En el entorno que he usado parece que los acentos del nombre de los documentos dan problemas.

Al especificar el directorio doc en el .gitattributes se soluciona el issue, ya que dicho directorio no hace falta que se descomprima en el vendor.

Ignorar este tipo de archivos y directorios (tests, documentación, etc.) se considera una buena práctica. Más info aquí: https://www.reddit.com/r/PHP/comments/2jzp6k/i_dont_need_your_tests_in_my_production/?rdt=33134

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.

2 participants