Return an array of loader objects from extract()

[insin]

This allows you to use: the output of extract() by not munging everything into a string (which use config should obviate the need for in Webpack 2).

It's far less surprising and makes it possible to provide plugins to preprocessor loaders directly via loader options instead of messing around with LoaderOptionsPlugin

This change is as per tmair/extract-text-webpack-plugin@1414036 - if @tmair wants to create their own PR for it, I'm happy to close this one.

[jsf-clabot]

All committers have signed the CLA.

[bebraw]

You need to sign the CLA. Apart from that, big 👍 from me.

[tmair]

@insin Go for it. I just did not know if my changes did break someting

[tmair]

@insin Travis is failing because of an old node version. If you rebase your canges to the current master the isse should be gone (See #335)

[codecov-io]

Current coverage is 86.40% (diff: 100%)

Merging #343 into master will increase coverage by 0.26%

@@             master       #343   diff @@
==========================================
  Files             4          4          
  Lines           332        331     -1   
  Methods          68         68          
  Messages          0          0          
  Branches         71         69     -2   
==========================================
  Hits            286        286          
+ Misses           46         45     -1   
  Partials          0          0          

Powered by Codecov. Last update c8494f3...e2f8fd8

[khankuan]

@insin thanks for the fix. Hopefully it can be rebased and merge soon. That's the main blocker I've found migrating to webpack2. Thanks!

[TheLarkInn]

A test would be a good addition to this PR.

[insin]

@TheLarkInn Added some unit tests for the current implementation and tweaked it slightly to use/support options.

Are there other use cases which extract() should support?

e.g. if you specify a loader as a string with query params, is it ok that it will be now passed straight through as-is, or is that something for Webpack itself to worry about?

[jakwuh]

Really waiting for this PR to be merged 👍

[joshwiens]

Thanks @insin - This will be in 2.0.0-rc.0 which will be published in short order