Skip to content
This repository has been archived by the owner on Feb 28, 2018. It is now read-only.

Move build-time npm packages to devDependencies #34

Merged
merged 1 commit into from
Feb 24, 2016

Conversation

ahoym
Copy link
Contributor

@ahoym ahoym commented Feb 24, 2016

Fixes: #33

I don't think this breaks any of the existing scripts, but it would probably be a good idea to double check.

The Context

The only run-time packages that the src/index.js file uses is 'react' and 'lodash.debounce'. Everything else can be installed as a devDependency.

Moving build-time packages like webpack, *-loaders, and babel-* helps reduce the cruft for users that want to npm shrinkwrap.

This PR

  • Moves the following packages from dependencies to devDependencies
    • autoprefixer-core
    • babel
    • babel-core
    • babel-jest
    • babel-loader
    • css-loader
    • csswring
    • extract-text-webpack-plugin
    • file-loader
    • gh-pages-deploy
    • jsx-loader
    • normalize.css
    • postcss-loader
    • root-require
    • sass-loader
    • script-loader
    • style-loader
    • url-loader
    • webpack
    • webpack-dev-server

The only run-time packages that the src/index.js file uses is 'react'
and 'lodash.debounce'. Everything else can be installed as a
devDependency.

This helps reduce the cruft for users that want to `npm shrinkwrap`.
@seethroughdev
Copy link
Owner

Thanks for doing this @ahoym . That's definitely how it should be.

seethroughdev pushed a commit that referenced this pull request Feb 24, 2016
Move build-time npm packages to devDependencies
@seethroughdev seethroughdev merged commit 9200fb4 into seethroughdev:master Feb 24, 2016
@ahoym ahoym deleted the organize-npm-packages branch February 24, 2016 22:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Organize npm dependencies
2 participants