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

If you start develop in a directory that doesn't contain a project it should warn you #23

Open
brent-hoover opened this issue Feb 3, 2022 · 9 comments
Assignees
Labels
bug Something isn't working hacktoberfest Tickets good for hackathon work

Comments

@brent-hoover
Copy link
Collaborator

Right now you just get:

 »» reaction develop api
reaction-cli: starting development on api: {"options":{}})
reaction-cli: Starting Mongo docker image
reaction-cli: Starting Open Commerce API Server in dev mode
[brent@ferret: 15:54:23 ~/Projects/merchstack/sample-data-plugin 
@brent-hoover brent-hoover added the bug Something isn't working label Feb 3, 2022
@brent-hoover brent-hoover added the hacktoberfest Tickets good for hackathon work label Oct 7, 2022
@breakerbar
Copy link
Contributor

I'm interested in working on this bug.

@delagroove
Copy link
Contributor

@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
basically adding the -S parameter: https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

@brent-hoover
Copy link
Collaborator Author

@breakerbar assigned this to you

@breakerbar
Copy link
Contributor

thanks - working on it now

@breakerbar
Copy link
Contributor

breakerbar commented Oct 14, 2022

@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:

$ reaction develop api
reaction-cli: Developing project: {"projectType":"api","options":{"debug":true,"mongoShutdown":true}}
reaction-cli: starting development on api: {"options":{"debug":true,"mongoShutdown":true}}
reaction-cli: Starting Mongo docker image
no configuration file provided: not found
npm
ERR! Missing script: "start:dev"
npm ERR!
npm ERR! To see a list of scripts, run:
npm ERR!   npm run

npm ERR! A complete log of this run can be found in:
npm ERR!     /Users/user-redacted/.npm/_logs/2022-10-14T19_25_46_677Z-debug.log

but otherwise you'd get this warning for an empty directory:

$ reaction develop api
reaction-cli: Developing project: {"projectType":"api","options":{"debug":true,"mongoShutdown":true}}
reaction-cli: It looks like you have not run `npm install` in this directory
reaction-cli: Please run `npm install` and try again

@brent-hoover
Copy link
Collaborator Author

@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.

@breakerbar
Copy link
Contributor

Thank you, very helpful. Will do.

@breakerbar
Copy link
Contributor

PR submitted ^

@DhanaSekharM
Copy link

Is this issue still open? I would like to work on this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working hacktoberfest Tickets good for hackathon work
Projects
None yet
Development

No branches or pull requests

4 participants