Skip to content
This repository has been archived by the owner on Aug 5, 2020. It is now read-only.

Proposal: helpful error for missing node_modules #922

Open
toddw opened this issue Apr 7, 2017 · 1 comment
Open

Proposal: helpful error for missing node_modules #922

toddw opened this issue Apr 7, 2017 · 1 comment

Comments

@toddw
Copy link
Contributor

toddw commented Apr 7, 2017

Enhancement Proposal

This enhancement is mostly helpful to people who are newer to the javascript ecosystem. I've noticed a lot of people having problems when they clone a GlueStick app but they forget to run yarn install or npm install before running the app. The most common error message they get is an error related to failing to spawn ./node_modules/.bin/gluestick which makes sense because it isn't there. Do you think it would be a good idea to check if the node_modules folder exists and if not we throw a more helpful error for these situations?

@zamotany
Copy link
Contributor

zamotany commented Apr 7, 2017

Yeah definitely. There are also other places where more user-friendly messages are needed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants