passy / autoprefixer-loader

Webpack loader for autoprefixer
MIT License
217 stars 25 forks source link

Shouldn't autoprefixer and postcss be peerDependencies? #33

Closed mik01aj closed 8 years ago

mik01aj commented 8 years ago

Currently your package.json has:

"dependencies": {
    "autoprefixer": "^6.0.2",
    "loader-utils": "^0.2.11",
    "postcss": "^5.0.4",
    "postcss-safe-parser": "^1.0.1"
  },

I think it would be better to put them into peerDependencies, so that autoprefixer-loader users could use any versions they need. Sass-loader does it this way.

Btw, if you have postcss-safe-parser, do you still need postcss dependency? ( postcss-safe-parser depends on postcss, so with npm@2 it gets installed twice.)

mik01aj commented 8 years ago

Btw. There is postcss-loader, so maybe this whole module could be deprecated.

ai commented 8 years ago

Yeap, Autoprefixer official way is to use only postcss-loader.