update install command to use `--save-dev`

open-cli-tools / concurrently

Run commands concurrently. Like `npm run watch-js & npm run watch-less` but better.

https://www.npmjs.com/package/concurrently

MIT License

7.11k stars 231 forks source link

Closed DetachHead closed 1 year ago

DetachHead commented 2 years ago

concurrently is a dev tool so it should be saved to devDependencies

paescuj commented 1 year ago

I'm in favor of this change for the following two reasons:

I assume that in a majority of projects "concurrently" is indeed used as a development tool.
It feels safer to propose the installation as devDependency by default.

I'm not saying that this is a good practise, but it's a valid use case for concurrently.

I also think this is a valid use case, but let's leave it to the users to declare "concurrently" as a productive dependency in this case.