refactor: disable source map using option

[alexander-akait]

Is it possible to disable generation source map generation using options? This will speed up compilation on watch when i disable source map in webpack. It would be nice if this option was taken from the webpack config by default. If your accept i can do PR.

[michael-ciniawsky]

@evilebottnawi Yep, right, could you check the #177, I intend to do that for v2.0.0 since it is technically a semver major, but wainting for other related loaders I need to update first style-loader/ extract-text-webpack-plugin

[alexander-akait]

@michael-ciniawsky can i help you with this?

[michael-ciniawsky]

@evilebottnawi Yep 😛 test if disabling sourcemaps works correctly

Sourcemap Implementation is here

v2.0.0 should work without css-loader and export a module directly (perf), but it is planned to export ES2015 Modules in the near future and style-loader/ extract-text-webpack-plugin don't support that yet. If you what to try that out you need to change this line to a CJS Export

- const module = `export default ${JSON.stringify(css)}`
+ const module = `module.exports = ${JSON.stringify(css)}`

webpack.config.js

const config = {
  module: {
    rules: [
      {
         test: /\.css$/,
         use: [ 'style-loader', 'postcss-loader' ]
      }
    ]
  }
}

Otherwise by using use: [ 'style-loader', 'css-loader', 'postcss-loader' ] you can skip that part, since postcss-loader will export a {String} as it is the current behaviour.

[michael-ciniawsky]

npm i -D michael-ciniawsky/postcss-loader#v2

[alexander-akait]

/cc @michael-ciniawsky let's do this, add options sourceMap as in css-loader?

[alexander-akait]

/cc @michael-ciniawsky ping 😄

[michael-ciniawsky]

Fixed by #191