queerviolet / bones

MIT License
36 stars 119 forks source link

Remove `nodemon` from default start (production) #25

Closed glebec closed 7 years ago

glebec commented 7 years ago

Following a conversation begun in #12.

Please let me know if I am wrong, but I am not entirely sure we should include nodemon in the default start script as that script will be used by Heroku — which, I believe, has its own process manager. Nodemon is a dev tool and I'd argue it belongs in devDependencies. In that case, we'd use nodemon in a start:dev script or similar, which the main dev script would use (instead of start).

Usage in less automatic platforms, e.g. Digital Ocean, would then use something like pm2 to manage the server instance.

glebec commented 7 years ago

Implemented in https://github.com/FullstackAcademy/bones/commit/f9715a14b02b552e2ac1d8e583069bf72539662a and https://github.com/FullstackAcademy/bones/commit/7107badb7c07081b8f6ecbd72690cb737cf6b3a2 together.