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 1943 #15

Merged
merged 4 commits into from Oct 8, 2019
Merged

Fix 1943 #15

merged 4 commits into from Oct 8, 2019

Conversation

@Mte90
Copy link
Member

@Mte90 Mte90 commented Sep 23, 2019

Remove node_modules folder if npm has errors on update, it is the best way to fix the issue.

Remove node_modules folder if npm has errors on update, it is the best way to fix the issue.
@Mte90
Copy link
Member Author

@Mte90 Mte90 commented Sep 23, 2019

Basically grunt crash because doesn't find a folder, I cannot thrust on the specific package or path. So I save the npm output on a specific file (because with pipe doesn't work), I search for ENOENT that is the error of missing file.
In case there is the error I remove the node_modules folder and download again.

@tomjn tomjn added the bug label Sep 23, 2019
provision/vvv-init.sh Show resolved Hide resolved
@Mte90 Mte90 requested a review from tomjn Oct 3, 2019
tomjn
tomjn approved these changes Oct 8, 2019
@tomjn tomjn merged commit dbf65fa into master Oct 8, 2019
@Mte90 Mte90 deleted the Mte90-fix-1943 branch Oct 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants