LD4P / sinopia_profile_editor

LD4P fork of lcdevnet profile-edit
Other
4 stars 1 forks source link

Isn't grunt an npm dev dependency, not a prod dependency? #17

Open ndushay opened 5 years ago

ndushay commented 5 years ago

See

https://github.com/ProjectMirador/research-and-demos/blob/master/minimal_redux_poc/package.json

note that babel and webpack are dev dependencies ...

If this is correct, do a PR for lcnetdev upstream repo (via LD4P/profile-edit branch then PR via github to upstream repo).

atz commented 5 years ago

grunt is a build dependency, afaict.

ndushay commented 5 years ago

yes, it's a build dependency, but not a run dependency, which is what I meant by "prod" -- once it's built, it's not needed