Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

fix for webpack 2 #312

Closed
wants to merge 2 commits into from
Closed

Conversation

freewayspb
Copy link

fix for use loaders with webpack2

@jsf-clabot
Copy link

jsf-clabot commented Dec 19, 2016

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

@codecov-io
Copy link

Current coverage is 85.84% (diff: 100%)

Merging #312 into master will not change coverage

@@             master       #312   diff @@
==========================================
  Files             4          4          
  Lines           332        332          
  Methods          68         68          
  Messages          0          0          
  Branches         71         71          
==========================================
  Hits            285        285          
  Misses           47         47          
  Partials          0          0          

Powered by Codecov. Last update 8232a3e...4984d8d

@@ -78,8 +78,8 @@ module.exports = {
...
module: {
loaders: [

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be rules now

@@ -78,8 +78,8 @@ module.exports = {
...
module: {
loaders: [
{ test: /\.scss$/i, loader: extractCSS.extract(['css','sass']) },
{ test: /\.less$/i, loader: extractLESS.extract(['css','less']) },
{ test: /\.scss$/i, loader: extractCSS.extract(['css-loader','sass-loader']) },
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should loader be use now as well?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Related: #265

The problem is that, if you use use:, you get a crash.

@michael-ciniawsky michael-ciniawsky mentioned this pull request Jan 23, 2017
9 tasks
@SpaceK33z
Copy link
Contributor

Closing in favor of #349.

@SpaceK33z SpaceK33z closed this Jan 23, 2017
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.

7 participants