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

Added yarn validator and nvm file #53

Merged
merged 1 commit into from
Dec 27, 2024

Conversation

FranciscoZapa
Copy link

Link al ticket
Sin ticket

Descripción del requerimiento
Se requiere crear un script para evitar ejecutar por error npm en lugar de yarn
Se requiere actualizar el readme para incluir la aclaracion de no usar npm para instalar dependencias
Se requiere agregar archivo nvm para especificar la version de node que debe usarse en este proyecto

Descripción de la solución
Se agrego el archivo .nvmrc
Se agrego el archivo preinstall-script.js que contiene el codigo para validar que se esta instalando dependencias con yarn
Se agrego el script preinstall-script.js a los al hook de presinstalacion de los scripts en package.json para validar que se haga con yarn y no npm
Se agrego una nota en el archivo readme para comunicar que se debe usar yarn y no npm al instalar dependencias

¿Cómo se puede probar?
Hacer npm i y validar que rompe y se muestra el mensaje de error
image

@colomfernando colomfernando merged commit 15ca147 into master Dec 27, 2024
4 checks passed
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