refactor: remove this.cacheable (#473)

evilebottnawi · joshwiens · commit 0d6d66de133a · 2017-06-13T22:05:29.000-05:00
Webpack 2 always cacheable all loaders by default.
diff --git a/lib/loader.js b/lib/loader.js
@@ -35,8 +35,6 @@ function sassLoader(content) {
         throw new Error("Synchronous compilation is not supported anymore. See https://github.com/webpack-contrib/sass-loader/issues/333");
     }
 
-    this.cacheable();
-
     const options = normalizeOptions(this, content, webpackImporter(
         resourcePath,
         pify(this.resolve.bind(this)),

Original file line number	Diff line number	Diff line change
`@@ -35,8 +35,6 @@ function sassLoader(content) {`
`35`	`35`	`throw new Error("Synchronous compilation is not supported anymore. See https://github.com/webpack-contrib/sass-loader/issues/333");`
`36`	`36`	`}`
`37`	`37`
`38`		`- this.cacheable();`
`39`		`-`
`40`	`38`	`const options = normalizeOptions(this, content, webpackImporter(`
`41`	`39`	`resourcePath,`
`42`	`40`	`pify(this.resolve.bind(this)),`