Fix linting

Author: ertrzyiks

gulpfile.js

@@ -42,4 +42,4 @@ gulp.task('test', ['build'], function () {
     return gulp.src('build/*.js', { read: false }).pipe(mocha());
 });
 
-gulp.task('default', ['test']);
+gulp.task('default', ['lint', 'test']);

index.js

@@ -2,14 +2,14 @@ var loaderUtils = require('loader-utils');
 var postcss     = require('postcss');
 var loadConfig = require('postcss-load-config');
 
-var postcssrc = null
+var postcssrc = null;
 
 function getPostCssRc() {
     if (!postcssrc) {
-        postcssrc = loadConfig()
+        postcssrc = loadConfig();
     }
 
-    return postcssrc
+    return postcssrc;
 }
 
 function PostCSSLoaderError(error) {
@@ -52,10 +52,10 @@ function getPluginsFromOptions(options, pack) {
 function loadConfigurationFromRc(callback) {
     getPostCssRc()
       .then(function (config) {
-          callback(null, {options: config.options, plugins: config.plugins});
+          callback(null, { options: config.options, plugins: config.plugins });
       })
-      .catch(function (err) {
-          callback(null, {options: {}, plugins: []});
+      .catch(function () {
+          callback(null, { options: {}, plugins: [] });
       });
 }
 
@@ -64,15 +64,18 @@ function getConfiguration(options, pack, callback) {
     // only if it's not provided explicitly
     if ( typeof options === 'undefined' ) {
         if ( pack ) {
-            callback(new Error('PostCSS plugin pack is not supported when configuration is read from file'));
+            callback(new Error(
+              'PostCSS plugin pack is not supported ' +
+              'when configuration is read from file'
+            ));
             return;
         }
 
         loadConfigurationFromRc(callback);
         return;
     }
 
-    var exec = options && options.exec
+    var exec = options && options.exec;
 
     if ( typeof options === 'function' ) {
         options = options.call(this, this);
@@ -87,7 +90,7 @@ function getConfiguration(options, pack, callback) {
         opts.syntax      = options.syntax;
     }
 
-    callback(null, {options: opts, plugins: plugins, exec: exec})
+    callback(null, { options: opts, plugins: plugins, exec: exec });
 }
 
 module.exports = function (source, map) {
@@ -97,7 +100,7 @@ module.exports = function (source, map) {
     var params = loaderUtils.parseQuery(this.query);
 
     var options  = this.options.postcss;
-    var pack = params.pack
+    var pack = params.pack;
     var loader   = this;
     var callback = this.async();
 
@@ -117,7 +120,7 @@ module.exports = function (source, map) {
                 inline:     params.sourceMap === 'inline',
                 annotation: false
             }
-        })
+        });
 
         if ( typeof map === 'string' ) map = JSON.parse(map);
         if ( map && map.mappings ) opts.map.prev = map;

test/test_common.js

@@ -15,7 +15,8 @@ describe('common aspects', function () {
     });
 
     it('lets other plugins alter the used plugins', function () {
-        var css = require('!raw-loader!../?rewrite=true!./support/cases/style.css');
+        var css = require('!raw-loader!../?rewrite=true!' +
+                          './support/cases/style.css');
         expect(css).to.eql('a { color: black }\n');
     });
 
@@ -34,7 +35,8 @@ describe('common aspects', function () {
     });
 
     it('inlines map', function () {
-        var css = require('!raw-loader!../?sourceMap=inline!./support/cases/style.css');
+        var css = require('!raw-loader!../?sourceMap=inline!' +
+                          './support/cases/style.css');
         expect(css).to.include('/*# sourceMappingURL=');
     });
 

test/with-options-loader/test.js

@@ -1,5 +1,3 @@
-var expect = require('chai').expect;
-
 describe('postcss-loader', function () {
 
     context('when used with options loader', function () {

test/with-options/test.js

@@ -6,9 +6,9 @@ describe('postcss-loader', function () {
         require('../test_common');
 
         it('processes CSS with custom plugins', function () {
-            var css = require('!raw-loader!../../?pack=blues!../support/cases/style.css');
+            var css = require('!raw-loader!../../?pack=blues!' +
+                              '../support/cases/style.css');
             expect(css).to.eql('a { color: blue }\n');
         });
-    })
-
+    });
 });