Enable lint option by default

Fixes #48

Author: simonsmith

CHANGELOG.md

@@ -1,5 +1,6 @@
 ### HEAD
 
+* Enable `lint` option by default
 * Add `postcss-apply`
 * Add `postcss-color-function`
 * Add `debug` option to pass an `postcss-debug` instance

README.md

@@ -17,7 +17,7 @@ Compiles CSS packages with:
 * [postcss-custom-media](https://github.com/postcss/postcss-custom-media)
 * [autoprefixer](https://github.com/postcss/autoprefixer)
 
-Each imported file is linted with [postcss-bem-linter](https://github.com/postcss/postcss-bem-linter) and minification is provided by [cssnano](http://cssnano.co/). Code style can also be checked with [stylelint](http://stylelint.io/)
+Each imported file is linted with [postcss-bem-linter](https://github.com/postcss/postcss-bem-linter) and [stylelint](http://stylelint.io/). Minification is provided by [cssnano](http://cssnano.co/).
 
 Additional plugins can be added via the configuration options.
 
@@ -45,7 +45,7 @@ Options:
   -h, --help                output usage information
   -V, --version             output the version number
   -m, --minify              minify output with cssnano
-  -l, --lint                ensure code adheres to the SUIT code style
+  -L, --no-lint             disable stylelint and postcss-bem-linter
   -i, --import-root [path]  the root directory for imported css files
   -c, --config [path]       a custom PostCSS config file
   -v, --verbose             log verbose output for debugging
@@ -128,7 +128,7 @@ function debug(plugins) {
 ##### `lint`
 
 * Type: `Boolean`
-* Default: `false`
+* Default: `true`
 
 Ensure code conforms to the [SUIT code style](https://github.com/suitcss/suit/blob/master/doc/STYLE.md)
 by using the [stylelint-config-suitcss](https://github.com/suitcss/stylelint-config-suitcss) package.
@@ -140,7 +140,6 @@ locally in your package.
 
 ```js
 {
-  lint: true,
   stylelint: {
     extends: 'stylelint-config-suitcss',
     rules: {

bin/suitcss

@@ -20,7 +20,7 @@ program
   .version(require('../package.json').version)
   .usage('[<input>] [<output>]')
   .option('-m, --minify', 'minify output with cssnano')
-  .option('-l, --lint', 'ensure code adheres to the SUIT code style')
+  .option('-L, --no-lint', 'disable stylelint and postcss-bem-linter')
   .option('-i, --import-root [path]', 'the root directory for imported css files')
   .option('-c, --config [path]', 'a custom PostCSS config file')
   .option('-v, --verbose', 'log verbose output for debugging')

lib/index.js

@@ -17,7 +17,7 @@ module.exports = preprocessor;
 
 var defaults = {
   debug: identity,
-  lint: false,
+  lint: true,
   minify: false,
   use: [
     'postcss-easy-import',

test/test.js

@@ -66,9 +66,9 @@ describe('suitcss', function() {
       }).catch(done);
     });
 
-    it('should allow stylelint to be enabled', function() {
-      var opts = mergeOptions({lint: true});
-      expect(opts.lint).to.be.true;
+    it('should allow stylelint to be disabled', function() {
+      var opts = mergeOptions({lint: false});
+      expect(opts.lint).to.be.false;
     });
 
     it('should allow a minify option to be set', function() {
@@ -131,7 +131,6 @@ describe('suitcss', function() {
     describe('using the transform option in postcss-import', function() {
       it('should use a default transform function that just returns the css', function(done) {
         suitcss('@import "./util.css";', {
-          lint: true,
           root: 'test/fixtures'
         }).then(function(result) {
           expect(result.css).to.equal('.u-img {\n  border-radius: 50%;\n}');
@@ -144,7 +143,6 @@ describe('suitcss', function() {
         var transformStub = sinon.stub().returns('body { color: blue; }');
 
         suitcss('@import "./util.css";', {
-          lint: true,
           root: 'test/fixtures',
           'postcss-easy-import': {
             transform: transformStub
@@ -160,7 +158,6 @@ describe('suitcss', function() {
 
       it('should also work with a promise returned from the custom transform function', function(done) {
         suitcss('@import "./util.css";', {
-          lint: true,
           root: 'test/fixtures',
           'postcss-easy-import': {
             transform: function() {
@@ -189,7 +186,6 @@ describe('suitcss', function() {
 
       it('should call the updateWatchTaskFiles function with the file paths', function(done) {
         suitcss('@import "./util.css";', {
-          lint: true,
           root: 'test/fixtures'
         }).then(function() {
           expect(updateWatchTaskFilesSpy.getCall(0).args[0][0]).to.contain('util.css');
@@ -202,7 +198,6 @@ describe('suitcss', function() {
         var onImportSpy = sinon.spy();
 
         suitcss('@import "./util.css";', {
-          lint: true,
           root: 'test/fixtures',
           'postcss-easy-import': {
             onImport: onImportSpy
@@ -259,7 +254,6 @@ describe('suitcss', function() {
       it('should lint the component', function() {
         return expect(
           suitcss(read('fixtures/component'), {
-            lint: true,
             root: 'test/fixtures',
             'postcss-reporter': {
               throwError: true
@@ -271,7 +265,6 @@ describe('suitcss', function() {
       it('should allow the config to be overidden', function() {
         return expect(
           suitcss('@import "./stylelint.css"', {
-            lint: true,
             root: 'test/fixtures',
             stylelint: {
               extends: 'stylelint-config-suitcss',
@@ -289,7 +282,6 @@ describe('suitcss', function() {
       it('should throw an error if stylelint fails', function() {
         return expect(
           suitcss('@import "./stylelint.css"', {
-            lint: true,
             root: 'test/fixtures',
             'postcss-reporter': {
               throwError: true