Fix bug: --no-color does note affect "FIXME!"

teppeis · Apr 4, 2014 · e91bce0 · e91bce0
1 parent 0fb0d26
commit e91bce0
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/lib/cli.js b/lib/cli.js
@@ -15,24 +15,24 @@ var Logger = function(enableColor, stdout, stderr) {
   this.stderr = stderr;
 };
 
-Logger.prototype.raw = function(msg) {
-  this.messages_.push(msg);
+Logger.prototype.raw = function(msg, opt_color) {
+  this.messages_.push(this.color_  && opt_color ? opt_color(msg) : msg);
 };
 
 Logger.prototype.info = function(msg) {
-  this.messages_.push(msg);
+  this.raw(msg);
 };
 
 Logger.prototype.warn = function(msg) {
-  this.messages_.push(this.color_ ? clc.yellow(msg) : msg);
+  this.raw(msg, clc.yellow);
 };
 
 Logger.prototype.error = function(msg) {
-  this.messages_.push(this.color_ ? clc.red(msg) : msg);
+  this.raw(msg, clc.red);
 };
 
 Logger.prototype.success = function(msg) {
-  this.messages_.push(this.color_ ? clc.green(msg) : msg);
+  this.raw(msg, clc.green);
 };
 
 Logger.prototype.items = function(items) {
@@ -231,7 +231,7 @@ function main(argv, stdout, stderr, exit) {
     if (needToFix) {
       if (program.fixInPlace) {
         fix(file, info);
-        log.raw(clc.cyanBright('FIXED!'));
+        log.raw(('FIXED!'), clc.cyan);
         fixed++;
       } else {
         log.error('FAIL!');
@@ -250,7 +250,7 @@ function main(argv, stdout, stderr, exit) {
   });
 
   var total = ok + ng + fixed;
-  log.raw('');
+  log.info('');
   if (ng) {
     log.error(ng + ' of ' + total + ' files failed');
     if (fixed) {