-
Notifications
You must be signed in to change notification settings - Fork 10
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
If you start develop in a directory that doesn't contain a project it should warn you #23
Comments
I'm interested in working on this bug. |
@breakerbar sure, you can fork the project, work on a solution and send a Pull Request which will be reviewed and merged if everything works well, remember to sign off your commits |
@breakerbar assigned this to you |
thanks - working on it now |
@zenweasel I'm unable to duplicate this behavior, can you provide a directory listing (or better yet a .zip/.tar.gz) of an example directory where this occurs? If I run this in the base directory of the project I get the following warning:
but otherwise you'd get this warning for an empty directory:
|
@breakerbar both the examples you provided are examples of it being broken. It should provide you an accurate message of what is wrong and how to fix it, not irrelevant or incorrect error messages. e.g. "You are attempting to start develop but are not in a project directory. Change to a project directory or create one". Specifically it should check for a package.json with one of the project types set. |
Thank you, very helpful. Will do. |
PR submitted ^ |
Is this issue still open? I would like to work on this |
Right now you just get:
The text was updated successfully, but these errors were encountered: