"lint" option implementation

Author: Misha Berezin

lib/cli.js

@@ -14,6 +14,7 @@ program
     .usage('[options] <file ...>')
     .option('-v, --verbose', 'verbose mode')
     .option('-c, --config [path]', 'configuration file path')
+    .option('-l, --lint', 'in case some fixes needed returns an error')
     .parse(process.argv);
 
 if (!program.args.length) {
@@ -26,21 +27,29 @@ var configPath = program.config || (process.cwd() + '/.csscomb.json');
 if (fs.existsSync(configPath)) {
     var comb = new Comb();
     var config = require(configPath);
-    var time = new Date();
+
+    console.time('spent');
+
     config.verbose = program.verbose === true || config.verbose;
+    config.lint = program.lint;
+
     comb.configure(config);
-    vow.all(program.args.map(function(path) {
-        return comb.processPath(path);
-    })).fail(function(e) {
-        console.log('stack: ', e.stack);
-        process.exit(1);
-    }).always(function() {
+
+    vow.all(program.args.map(comb.processPath.bind(comb)))
+    .then(function() {
         if (config.verbose) {
             console.log('');
             console.log(comb.processed + ' file' + (comb.processed === 1 ? '' : 's') + ' processed');
             console.log(comb.changed + ' file' + (comb.changed === 1 ? '' : 's') + ' fixed');
-            console.log((new Date().getTime() - time.getTime()) + ' ms spent');
+            console.timeEnd('spent');
+        }
+        if (config.lint && comb.tbchanged) {
+            process.exit(1);
         }
+    })
+    .fail(function(e) {
+        console.log('stack: ', e.stack);
+        process.exit(1);
     });
 } else {
     console.log('Configuration file ' + configPath + ' was not found.');

lib/csscomb.js

@@ -2,6 +2,7 @@ var gonzales = require('gonzales');
 var minimatch = require('minimatch');
 var vow = require('vow');
 var vfs = require('vow-fs');
+var doNothing = function() {};
 
 /**
  * Starts Code Style processing process.
@@ -56,8 +57,10 @@ Comb.prototype = {
         });
 
         this.processed = 0;
+        this.tbchanged = 0;
         this.changed = 0;
         this._verbose = config.verbose;
+        this._lint = config.lint;
     },
 
     /**
@@ -125,11 +128,22 @@ Comb.prototype = {
             return vfs.read(path, 'utf8').then(function(data) {
                 var processedData = _this.processString(data, path);
                 var changed = data !== processedData;
-                return vfs.write(path, processedData, 'utf8').then(function() {
-                    _this.processed++;
-                    if (changed) _this.changed++;
-                    if (_this._verbose) console.log((changed ? '✓' : ' ') + ' ' + path);
-                });
+                var lint = _this._lint;
+
+                var tick = (changed) ? (lint ? '\x1B[31m✓\x1B[39m' : '✓') : ' ';
+                var message = _this._verbose ? console.log.bind(null, tick, path) : doNothing;
+
+                _this.processed++;
+                changed && _this.tbchanged++;
+
+                if (!changed || lint) {
+                    message();
+                } else {
+                    return vfs.write(path, processedData, 'utf8').then(function() {
+                        _this.changed++;
+                        message();
+                    });
+                }
             });
         }
         return null;