Move all gulp/postcss dependencies to devDependencies #111
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Currently, this repo's node dependencies (for building CSS) are sprinkled across
dependencies
anddevDependencies
, when all of them should arguably be indevDependences
- they're required when working within this repo to build CSS, but they're not required by anything externally consuming this repo since the built CSS is already included.This will avoid compelling external consumers of this repo to install postcss, gulp, and a bunch of plugins.
I tested blowing away
node_modules
then re-runningnpm i
andnpm run build-css
to confirm no difference in behavior within this repo.