-
Notifications
You must be signed in to change notification settings - Fork 81
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
Why use --save-dev flag to install video.js? #126
Comments
In the other repo would work ok as well. Either way works, I think the idea behind |
Hi! I just asked myself the same question. In my team, we publish a library of components, one of them being a Player component which wraps video.js. I will definitely agree with @gkatsev here that it is a bit of a grey area. For instance, in my case, our library is bundled with rollup.js and our apps built with webpack. It does not seem to matter whether I put video.js in dependencies or devDependencies as in both cases our applications still works fine. The final bundle size remains the same too. Since it doesn't seem to matter. I think for clarity, I'll move the video.js into the dependencies array to be inline with the NPM docs
Thank you |
The documentation doesn't explain why the --save-dev flag is used instead of normal install. If this is going to be on the final project output then it should be in the dependencies and not on deveDependencies or there is some reason you recommend this?
The text was updated successfully, but these errors were encountered: