Add ESLint and lint all files; add lint check to CI

.eslintrc.json

@@ -0,0 +1,11 @@
+{
+    "extends": "google",
+    "rules": {
+      "no-var": "off",
+      "prefer-const": "off",
+      "eol-last": ["error", "always"],
+      "require-jsdoc": "off",
+      "guard-for-in": "off",
+      "prefer-rest-params": "off"
+    }
+}

.gitignore

@@ -1 +1,2 @@
 **/*.sw*
+node_modules

.travis.yml

@@ -10,3 +10,7 @@ node_js:
   - "0.11"
   - "0.10"
   - "0.8"
+script:
+  - npm install
+  - "./node_modules/.bin/eslint . "
+  - npm test

examples/normal-usage.js

@@ -1,34 +1,36 @@
 var colors = require('../lib/index');
 
-console.log("First some yellow text".yellow);
+console.log('First some yellow text'.yellow);
 
-console.log("Underline that text".yellow.underline);
+console.log('Underline that text'.yellow.underline);
 
-console.log("Make it bold and red".red.bold);
+console.log('Make it bold and red'.red.bold);
 
-console.log(("Double Raindows All Day Long").rainbow)
+console.log(('Double Raindows All Day Long').rainbow);
 
-console.log("Drop the bass".trap)
+console.log('Drop the bass'.trap);
 
-console.log("DROP THE RAINBOW BASS".trap.rainbow)
+console.log('DROP THE RAINBOW BASS'.trap.rainbow);
 
+// styles not widely supported
+console.log('Chains are also cool.'.bold.italic.underline.red);
 
-console.log('Chains are also cool.'.bold.italic.underline.red); // styles not widely supported
+// styles not widely supported
-
+console.log('So '.green + 'are'.underline + ' ' + 'inverse'.inverse
-console.log('So '.green + 'are'.underline + ' ' + 'inverse'.inverse + ' styles! '.yellow.bold); // styles not widely supported
+  + ' styles! '.yellow.bold);
-console.log("Zebras are so fun!".zebra);
+console.log('Zebras are so fun!'.zebra);
 
 //
 // Remark: .strikethrough may not work with Mac OS Terminal App
 //
-console.log("This is " + "not".strikethrough + " fun.");
+console.log('This is ' + 'not'.strikethrough + ' fun.');
 
-console.log('Background color attack!'.black.bgWhite)
+console.log('Background color attack!'.black.bgWhite);
-console.log('Use random styles on everything!'.random)
+console.log('Use random styles on everything!'.random);
-console.log('America, Heck Yeah!'.america)
+console.log('America, Heck Yeah!'.america);
 
 
-console.log('Setting themes is useful')
+console.log('Setting themes is useful');
 
 //
 // Custom themes
@@ -45,17 +47,17 @@ colors.setTheme({
   help: 'cyan',
   warn: 'yellow',
   debug: 'blue',
-  error: 'red'
+  error: 'red',
 });
 
 // outputs red text
-console.log("this is an error".error);
+console.log('this is an error'.error);
 
 // outputs yellow text
-console.log("this is a warning".warn);
+console.log('this is a warning'.warn);
 
 // outputs grey text
-console.log("this is an input".input);
+console.log('this is an input'.input);
 
 console.log('Generic logging theme as file'.green.bold.underline);
 
@@ -67,12 +69,13 @@ try {
 }
 
 // outputs red text
-console.log("this is an error".error);
+console.log('this is an error'.error);
 
 // outputs yellow text
-console.log("this is a warning".warn);
+console.log('this is a warning'.warn);
 
 // outputs grey text
-console.log("this is an input".input);
+console.log('this is an input'.input);
+
+// console.log("Don't summon".zalgo)
 
-//console.log("Don't summon".zalgo)

examples/safe-string.js

@@ -1,37 +1,39 @@
 var colors = require('../safe');
 
-console.log(colors.yellow("First some yellow text"));
+console.log(colors.yellow('First some yellow text'));
 
-console.log(colors.yellow.underline("Underline that text"));
+console.log(colors.yellow.underline('Underline that text'));
 
-console.log(colors.red.bold("Make it bold and red"));
+console.log(colors.red.bold('Make it bold and red'));
 
-console.log(colors.rainbow("Double Raindows All Day Long"))
+console.log(colors.rainbow('Double Raindows All Day Long'));
 
-console.log(colors.trap("Drop the bass"))
+console.log(colors.trap('Drop the bass'));
 
-console.log(colors.rainbow(colors.trap("DROP THE RAINBOW BASS")));
+console.log(colors.rainbow(colors.trap('DROP THE RAINBOW BASS')));
 
-console.log(colors.bold.italic.underline.red('Chains are also cool.')); // styles not widely supported
+// styles not widely supported
+console.log(colors.bold.italic.underline.red('Chains are also cool.'));
 
+// styles not widely supported
+console.log(colors.green('So ') + colors.underline('are') + ' '
+  + colors.inverse('inverse') + colors.yellow.bold(' styles! '));
 
-console.log(colors.green('So ') + colors.underline('are') + ' ' + colors.inverse('inverse') + colors.yellow.bold(' styles! ')); // styles not widely supported
+console.log(colors.zebra('Zebras are so fun!'));
 
-console.log(colors.zebra("Zebras are so fun!"));
+console.log('This is ' + colors.strikethrough('not') + ' fun.');
-
-console.log("This is " + colors.strikethrough("not") + " fun.");
 
 
 console.log(colors.black.bgWhite('Background color attack!'));
-console.log(colors.random('Use random styles on everything!'))
+console.log(colors.random('Use random styles on everything!'));
 console.log(colors.america('America, Heck Yeah!'));
 
-console.log('Setting themes is useful')
+console.log('Setting themes is useful');
 
 //
 // Custom themes
 //
-//console.log('Generic logging theme as JSON'.green.bold.underline);
+// console.log('Generic logging theme as JSON'.green.bold.underline);
 // Load theme with JSON literal
 colors.setTheme({
   silly: 'rainbow',
@@ -43,17 +45,17 @@ colors.setTheme({
   help: 'cyan',
   warn: 'yellow',
   debug: 'blue',
-  error: 'red'
+  error: 'red',
 });
 
 // outputs red text
-console.log(colors.error("this is an error"));
+console.log(colors.error('this is an error'));
 
 // outputs yellow text
-console.log(colors.warn("this is a warning"));
+console.log(colors.warn('this is a warning'));
 
 // outputs grey text
-console.log(colors.input("this is an input"));
+console.log(colors.input('this is an input'));
 
 
 // console.log('Generic logging theme as file'.green.bold.underline);
@@ -62,15 +64,14 @@ console.log(colors.input("this is an input"));
 colors.setTheme(__dirname + '/../themes/generic-logging.js');
 
 // outputs red text
-console.log(colors.error("this is an error"));
+console.log(colors.error('this is an error'));
 
 // outputs yellow text
-console.log(colors.warn("this is a warning"));
+console.log(colors.warn('this is a warning'));
 
 // outputs grey text
-console.log(colors.input("this is an input"));
+console.log(colors.input('this is an input'));
 
 // console.log(colors.zalgo("Don't summon him"))
 
 
-

lib/colors.js

@@ -40,38 +40,38 @@ var newLineRegex = new RegExp(/[\r\n]+/g);
 
 colors.supportsColor = require('./system/supports-colors').supportsColor;
 
-if (typeof colors.enabled === "undefined") {
+if (typeof colors.enabled === 'undefined') {
   colors.enabled = colors.supportsColor() !== false;
 }
 
-colors.enable = function () {
+colors.enable = function() {
   colors.enabled = true;
 };
 
-colors.disable = function () {
+colors.disable = function() {
   colors.enabled = false;
 };
 
-colors.stripColors = colors.strip = function(str){
+colors.stripColors = colors.strip = function(str) {
-  return ("" + str).replace(/\x1B\[\d+m/g, '');
+  return ('' + str).replace(/\x1B\[\d+m/g, '');
 };
 
-
+// eslint-disable-next-line no-unused-vars
-var stylize = colors.stylize = function stylize (str, style) {
+var stylize = colors.stylize = function stylize(str, style) {
   if (!colors.enabled) {
     return str+'';
   }
 
   return ansiStyles[style].open + str + ansiStyles[style].close;
-}
+};
 
 var matchOperatorsRe = /[|\\{}()[\]^$+*?.]/g;
-var escapeStringRegexp = function (str) {
+var escapeStringRegexp = function(str) {
   if (typeof str !== 'string') {
     throw new TypeError('Expected a string');
   }
-  return str.replace(matchOperatorsRe,  '\\$&');
+  return str.replace(matchOperatorsRe, '\\$&');
-}
+};
 
 function build(_styles) {
   var builder = function builder() {
@@ -84,15 +84,16 @@ function build(_styles) {
   return builder;
 }
 
-var styles = (function () {
+var styles = (function() {
   var ret = {};
   ansiStyles.grey = ansiStyles.gray;
-  Object.keys(ansiStyles).forEach(function (key) {
+  Object.keys(ansiStyles).forEach(function(key) {
-    ansiStyles[key].closeRe = new RegExp(escapeStringRegexp(ansiStyles[key].close), 'g');
+    ansiStyles[key].closeRe =
+      new RegExp(escapeStringRegexp(ansiStyles[key].close), 'g');
     ret[key] = {
-      get: function () {
+      get: function() {
         return build(this._styles.concat(key));
-      }
+      },
     };
   });
   return ret;
@@ -119,7 +120,7 @@ function applyStyle() {
   while (i--) {
     var code = ansiStyles[nestedStyles[i]];
     str = code.open + str.replace(code.closeRe, code.open) + code.close;
-    if (newLinesPresent){
+    if (newLinesPresent) {
       str = str.replace(newLineRegex, code.close + '\n' + code.open);
     }
   }
@@ -127,46 +128,49 @@ function applyStyle() {
   return str;
 }
 
-colors.setTheme = function (theme) {
+colors.setTheme = function(theme) {
   if (typeof theme === 'string') {
     console.log('colors.setTheme now only accepts an object, not a string.  ' +
-      'If you are trying to set a theme from a file, it is now your (the caller\'s) responsibility to require the file.  ' +
+      'If you are trying to set a theme from a file, it is now your (the ' +
-      'The old syntax looked like colors.setTheme(__dirname + \'/../themes/generic-logging.js\'); ' +
+      'caller\'s) responsibility to require the file.  The old syntax ' +
-      'The new syntax looks like colors.setTheme(require(__dirname + \'/../themes/generic-logging.js\'));');
+      'looked like colors.setTheme(__dirname + ' +
+      '\'/../themes/generic-logging.js\'); The new syntax looks like '+
+      'colors.setTheme(require(__dirname + ' +
+      '\'/../themes/generic-logging.js\'));');
     return;
   }
   for (var style in theme) {
-    (function(style){
+    (function(style) {
-      colors[style] = function(str){
+      colors[style] = function(str) {
-        if (typeof theme[style] === 'object'){
+        if (typeof theme[style] === 'object') {
           var out = str;
-          for (var i in theme[style]){
+          for (var i in theme[style]) {
             out = colors[theme[style][i]](out);
           }
           return out;
         }
         return colors[theme[style]](str);
       };
-    })(style)
+    })(style);
   }
-}
+};
 
 function init() {
   var ret = {};
-  Object.keys(styles).forEach(function (name) {
+  Object.keys(styles).forEach(function(name) {
     ret[name] = {
-      get: function () {
+      get: function() {
         return build([name]);
-      }
+      },
     };
   });
   return ret;
 }
 
-var sequencer = function sequencer (map, str) {
+var sequencer = function sequencer(map, str) {
-  var exploded = str.split(""), i = 0;
+  var exploded = str.split('');
   exploded = exploded.map(map);
-  return exploded.join("");
+  return exploded.join('');
 };
 
 // custom formatter methods
@@ -178,14 +182,14 @@ colors.maps = {};
 colors.maps.america = require('./maps/america');
 colors.maps.zebra = require('./maps/zebra');
 colors.maps.rainbow = require('./maps/rainbow');
-colors.maps.random = require('./maps/random')
+colors.maps.random = require('./maps/random');
 
 for (var map in colors.maps) {
-  (function(map){
+  (function(map) {
-    colors[map] = function (str) {
+    colors[map] = function(str) {
       return sequencer(colors.maps[map], str);
-    }
+    };
-  })(map)
+  })(map);
 }
 
 defineProps(colors, init());

lib/custom/trap.js

@@ -1,45 +1,46 @@
-module['exports'] = function runTheTrap (text, options) {
+module['exports'] = function runTheTrap(text, options) {
-  var result = "";
+  var result = '';
-  text = text || "Run the trap, drop the bass";
+  text = text || 'Run the trap, drop the bass';
   text = text.split('');
   var trap = {
-    a: ["\u0040", "\u0104", "\u023a", "\u0245", "\u0394", "\u039b", "\u0414"],
+    a: ['\u0040', '\u0104', '\u023a', '\u0245', '\u0394', '\u039b', '\u0414'],
-    b: ["\u00df", "\u0181", "\u0243", "\u026e", "\u03b2", "\u0e3f"],
+    b: ['\u00df', '\u0181', '\u0243', '\u026e', '\u03b2', '\u0e3f'],
-    c: ["\u00a9", "\u023b", "\u03fe"],
+    c: ['\u00a9', '\u023b', '\u03fe'],
-    d: ["\u00d0", "\u018a", "\u0500" , "\u0501" ,"\u0502", "\u0503"],
+    d: ['\u00d0', '\u018a', '\u0500', '\u0501', '\u0502', '\u0503'],
-    e: ["\u00cb", "\u0115", "\u018e", "\u0258", "\u03a3", "\u03be", "\u04bc", "\u0a6c"],
+    e: ['\u00cb', '\u0115', '\u018e', '\u0258', '\u03a3', '\u03be', '\u04bc',
-    f: ["\u04fa"],
+         '\u0a6c'],
-    g: ["\u0262"],
+    f: ['\u04fa'],
-    h: ["\u0126", "\u0195", "\u04a2", "\u04ba", "\u04c7", "\u050a"],
+    g: ['\u0262'],
-    i: ["\u0f0f"],
+    h: ['\u0126', '\u0195', '\u04a2', '\u04ba', '\u04c7', '\u050a'],
-    j: ["\u0134"],
+    i: ['\u0f0f'],
-    k: ["\u0138", "\u04a0", "\u04c3", "\u051e"],
+    j: ['\u0134'],
-    l: ["\u0139"],
+    k: ['\u0138', '\u04a0', '\u04c3', '\u051e'],
-    m: ["\u028d", "\u04cd", "\u04ce", "\u0520", "\u0521", "\u0d69"],
+    l: ['\u0139'],
-    n: ["\u00d1", "\u014b", "\u019d", "\u0376", "\u03a0", "\u048a"],
+    m: ['\u028d', '\u04cd', '\u04ce', '\u0520', '\u0521', '\u0d69'],
-    o: ["\u00d8", "\u00f5", "\u00f8", "\u01fe", "\u0298", "\u047a", "\u05dd", "\u06dd", "\u0e4f"],
+    n: ['\u00d1', '\u014b', '\u019d', '\u0376', '\u03a0', '\u048a'],
-    p: ["\u01f7", "\u048e"],
+    o: ['\u00d8', '\u00f5', '\u00f8', '\u01fe', '\u0298', '\u047a', '\u05dd',
-    q: ["\u09cd"],
+         '\u06dd', '\u0e4f'],
-    r: ["\u00ae", "\u01a6", "\u0210", "\u024c", "\u0280", "\u042f"],
+    p: ['\u01f7', '\u048e'],
-    s: ["\u00a7", "\u03de", "\u03df", "\u03e8"],
+    q: ['\u09cd'],
-    t: ["\u0141", "\u0166", "\u0373"],
+    r: ['\u00ae', '\u01a6', '\u0210', '\u024c', '\u0280', '\u042f'],
-    u: ["\u01b1", "\u054d"],
+    s: ['\u00a7', '\u03de', '\u03df', '\u03e8'],
-    v: ["\u05d8"],
+    t: ['\u0141', '\u0166', '\u0373'],
-    w: ["\u0428", "\u0460", "\u047c", "\u0d70"],
+    u: ['\u01b1', '\u054d'],
-    x: ["\u04b2", "\u04fe", "\u04fc", "\u04fd"],
+    v: ['\u05d8'],
-    y: ["\u00a5", "\u04b0", "\u04cb"],
+    w: ['\u0428', '\u0460', '\u047c', '\u0d70'],
-    z: ["\u01b5", "\u0240"]
+    x: ['\u04b2', '\u04fe', '\u04fc', '\u04fd'],
-  }
+    y: ['\u00a5', '\u04b0', '\u04cb'],
-  text.forEach(function(c){
+    z: ['\u01b5', '\u0240'],
+  };
+  text.forEach(function(c) {
     c = c.toLowerCase();
-    var chars = trap[c] || [" "];
+    var chars = trap[c] || [' '];
     var rand = Math.floor(Math.random() * chars.length);
-    if (typeof trap[c] !== "undefined") {
+    if (typeof trap[c] !== 'undefined') {
       result += trap[c][rand];
     } else {
       result += c;
     }
   });
   return result;
-
+};
-}

lib/custom/zalgo.js

@@ -1,8 +1,8 @@
 // please no
 module['exports'] = function zalgo(text, options) {
-  text = text || "   he is here   ";
+  text = text || '   he is here   ';
   var soul = {
-    "up" : [
+    'up': [
       '̍', '̎', '̄', '̅',
       '̿', '̑', '̆', '̐',
       '͒', '͗', '͑', '̇',
@@ -15,9 +15,9 @@ module['exports'] = function zalgo(text, options) {
       'ͦ', 'ͧ', 'ͨ', 'ͩ',
       'ͪ', 'ͫ', 'ͬ', 'ͭ',
       'ͮ', 'ͯ', '̾', '͛',
-      '͆', '̚'
+      '͆', '̚',
     ],
-    "down" : [
+    'down': [
       '̖', '̗', '̘', '̙',
       '̜', '̝', '̞', '̟',
       '̠', '̤', '̥', '̦',
@@ -27,28 +27,27 @@ module['exports'] = function zalgo(text, options) {
       '̺', '̻', '̼', 'ͅ',
       '͇', '͈', '͉', '͍',
       '͎', '͓', '͔', '͕',
-      '͖', '͙', '͚', '̣'
+      '͖', '͙', '͚', '̣',
     ],
-    "mid" : [
+    'mid': [
       '̕', '̛', '̀', '́',
       '͘', '̡', '̢', '̧',
       '̨', '̴', '̵', '̶',
       '͜', '͝', '͞',
       '͟', '͠', '͢', '̸',
-      '̷', '͡', ' ҉'
+      '̷', '͡', ' ҉',
-    ]
+    ],
-  },
+  };
-  all = [].concat(soul.up, soul.down, soul.mid),
+  var all = [].concat(soul.up, soul.down, soul.mid);
-  zalgo = {};
 
   function randomNumber(range) {
     var r = Math.floor(Math.random() * range);
     return r;
   }
 
-  function is_char(character) {
+  function isChar(character) {
     var bool = false;
-    all.filter(function (i) {
+    all.filter(function(i) {
       bool = (i === character);
     });
     return bool;
@@ -56,19 +55,25 @@ module['exports'] = function zalgo(text, options) {
 
 
   function heComes(text, options) {
-    var result = '', counts, l;
+    var result = '';
+    var counts;
+    var l;
     options = options || {};
-    options["up"] =   typeof options["up"]   !== 'undefined' ? options["up"]   : true;
+    options['up'] =
-    options["mid"] =  typeof options["mid"]  !== 'undefined' ? options["mid"]  : true;
+      typeof options['up'] !== 'undefined' ? options['up'] : true;
-    options["down"] = typeof options["down"] !== 'undefined' ? options["down"] : true;
+    options['mid'] =
-    options["size"] = typeof options["size"] !== 'undefined' ? options["size"] : "maxi";
+      typeof options['mid'] !== 'undefined' ? options['mid'] : true;
+    options['down'] =
+      typeof options['down'] !== 'undefined' ? options['down'] : true;
+    options['size'] =
+      typeof options['size'] !== 'undefined' ? options['size'] : 'maxi';
     text = text.split('');
     for (l in text) {
-      if (is_char(l)) {
+      if (isChar(l)) {
         continue;
       }
       result = result + text[l];
-      counts = {"up" : 0, "down" : 0, "mid" : 0};
+      counts = {'up': 0, 'down': 0, 'mid': 0};
       switch (options.size) {
       case 'mini':
         counts.up = randomNumber(8);
@@ -87,10 +92,10 @@ module['exports'] = function zalgo(text, options) {
         break;
       }
 
-      var arr = ["up", "mid", "down"];
+      var arr = ['up', 'mid', 'down'];
       for (var d in arr) {
         var index = arr[d];
-        for (var i = 0 ; i <= counts[index]; i++) {
+        for (var i = 0; i <= counts[index]; i++) {
           if (options[index]) {
             result = result + soul[index][randomNumber(soul[index].length)];
           }
@@ -101,4 +106,5 @@ module['exports'] = function zalgo(text, options) {
   }
   // don't summon him
   return heComes(text, options);
-}
+};
+

lib/extendStringPrototype.js

@@ -1,43 +1,42 @@
 var colors = require('./colors');
 
-module['exports'] = function () {
+module['exports'] = function() {
-
   //
   // Extends prototype of native string object to allow for "foo".red syntax
   //
-  var addProperty = function (color, func) {
+  var addProperty = function(color, func) {
     String.prototype.__defineGetter__(color, func);
   };
 
-  addProperty('strip', function () {
+  addProperty('strip', function() {
     return colors.strip(this);
   });
 
-  addProperty('stripColors', function () {
+  addProperty('stripColors', function() {
     return colors.strip(this);
   });
 
-  addProperty("trap", function(){
+  addProperty('trap', function() {
     return colors.trap(this);
   });
 
-  addProperty("zalgo", function(){
+  addProperty('zalgo', function() {
     return colors.zalgo(this);
   });
 
-  addProperty("zebra", function(){
+  addProperty('zebra', function() {
     return colors.zebra(this);
   });
 
-  addProperty("rainbow", function(){
+  addProperty('rainbow', function() {
     return colors.rainbow(this);
   });
 
-  addProperty("random", function(){
+  addProperty('random', function() {
     return colors.random(this);
   });
 
-  addProperty("america", function(){
+  addProperty('america', function() {
     return colors.america(this);
   });
 
@@ -45,8 +44,8 @@ module['exports'] = function () {
   // Iterate through all default styles and colors
   //
   var x = Object.keys(colors.styles);
-  x.forEach(function (style) {
+  x.forEach(function(style) {
-    addProperty(style, function () {
+    addProperty(style, function() {
       return colors.stylize(this, style);
     });
   });
@@ -57,25 +56,28 @@ module['exports'] = function () {
     // on String that you should not overwrite.
     //
     var stringPrototypeBlacklist = [
-      '__defineGetter__', '__defineSetter__', '__lookupGetter__', '__lookupSetter__', 'charAt', 'constructor',
+      '__defineGetter__', '__defineSetter__', '__lookupGetter__',
-      'hasOwnProperty', 'isPrototypeOf', 'propertyIsEnumerable', 'toLocaleString', 'toString', 'valueOf', 'charCodeAt',
+      '__lookupSetter__', 'charAt', 'constructor', 'hasOwnProperty',
-      'indexOf', 'lastIndexOf', 'length', 'localeCompare', 'match', 'repeat', 'replace', 'search', 'slice', 'split', 'substring',
+      'isPrototypeOf', 'propertyIsEnumerable', 'toLocaleString', 'toString',
-      'toLocaleLowerCase', 'toLocaleUpperCase', 'toLowerCase', 'toUpperCase', 'trim', 'trimLeft', 'trimRight'
+      'valueOf', 'charCodeAt', 'indexOf', 'lastIndexOf', 'length',
+      'localeCompare', 'match', 'repeat', 'replace', 'search', 'slice',
+      'split', 'substring', 'toLocaleLowerCase', 'toLocaleUpperCase',
+      'toLowerCase', 'toUpperCase', 'trim', 'trimLeft', 'trimRight',
     ];
 
-    Object.keys(theme).forEach(function (prop) {
+    Object.keys(theme).forEach(function(prop) {
       if (stringPrototypeBlacklist.indexOf(prop) !== -1) {
-        console.log('warn: '.red + ('String.prototype' + prop).magenta + ' is probably something you don\'t want to override. Ignoring style name');
+        console.log('warn: '.red + ('String.prototype' + prop).magenta +
-      }
+          ' is probably something you don\'t want to override.  ' +
-      else {
+          'Ignoring style name');
+      } else {
         if (typeof(theme[prop]) === 'string') {
           colors[prop] = colors[theme[prop]];
-          addProperty(prop, function () {
+          addProperty(prop, function() {
             return colors[theme[prop]](this);
           });
-        }
+        } else {
-        else {
+          addProperty(prop, function() {
-          addProperty(prop, function () {
             var ret = this;
             for (var t = 0; t < theme[prop].length; t++) {
               ret = colors[theme[prop][t]](ret);
@@ -87,7 +89,7 @@ module['exports'] = function () {
     });
   }
 
-  colors.setTheme = function (theme) {
+  colors.setTheme = function(theme) {
     if (typeof theme === 'string') {
       try {
         colors.themes[theme] = require(theme);
@@ -101,5 +103,4 @@ module['exports'] = function () {
       applyTheme(theme);
     }
   };
-
 };

lib/index.js

@@ -1,9 +1,10 @@
 var colors = require('./colors');
 module['exports'] = colors;
 
-// Remark: By default, colors will add style properties to String.prototype
+// Remark: By default, colors will add style properties to String.prototype.
 //
-// If you don't wish to extend String.prototype you can do this instead and native String will not be touched
+// If you don't wish to extend String.prototype, you can do this instead and
+// native String will not be touched:
 //
 //   var colors = require('colors/safe);
 //   colors.red("foo")

lib/maps/america.js

@@ -1,12 +1,12 @@
 var colors = require('../colors');
 
 module['exports'] = (function() {
-  return function (letter, i, exploded) {
+  return function(letter, i, exploded) {
-    if(letter === " ") return letter;
+    if (letter === ' ') return letter;
-    switch(i%3) {
+    switch (i%3) {
       case 0: return colors.red(letter);
-      case 1: return colors.white(letter)
+      case 1: return colors.white(letter);
-      case 2: return colors.blue(letter)
+      case 2: return colors.blue(letter);
     }
-  }
+  };
 })();

lib/maps/rainbow.js

@@ -1,9 +1,10 @@
 var colors = require('../colors');
 
-module['exports'] = (function () {
+module['exports'] = (function() {
-  var rainbowColors = ['red', 'yellow', 'green', 'blue', 'magenta']; //RoY G BiV
+  // RoY G BiV
-  return function (letter, i, exploded) {
+  var rainbowColors = ['red', 'yellow', 'green', 'blue', 'magenta'];
-    if (letter === " ") {
+  return function(letter, i, exploded) {
+    if (letter === ' ') {
       return letter;
     } else {
       return colors[rainbowColors[i++ % rainbowColors.length]](letter);

lib/maps/random.js

@@ -1,8 +1,12 @@
 var colors = require('../colors');
 
-module['exports'] = (function () {
+module['exports'] = (function() {
-  var available = ['underline', 'inverse', 'grey', 'yellow', 'red', 'green', 'blue', 'white', 'cyan', 'magenta'];
+  var available = ['underline', 'inverse', 'grey', 'yellow', 'red', 'green',
+    'blue', 'white', 'cyan', 'magenta'];
   return function(letter, i, exploded) {
-    return letter === " " ? letter : colors[available[Math.round(Math.random() * (available.length - 1))]](letter);
+    return letter === ' ' ? letter :
+      colors[
+        available[Math.round(Math.random() * (available.length - 2))]
+      ](letter);
   };
 })();

lib/maps/zebra.js

@@ -1,5 +1,5 @@
 var colors = require('../colors');
 
-module['exports'] = function (letter, i, exploded) {
+module['exports'] = function(letter, i, exploded) {
   return i % 2 === 0 ? letter : colors.inverse(letter);
 };

lib/styles.js

@@ -65,11 +65,11 @@ var codes = {
   blueBG: [44, 49],
   magentaBG: [45, 49],
   cyanBG: [46, 49],
-  whiteBG: [47, 49]
+  whiteBG: [47, 49],
 
 };
 
-Object.keys(codes).forEach(function (key) {
+Object.keys(codes).forEach(function(key) {
   var val = codes[key];
   var style = styles[key] = [];
   style.open = '\u001b[' + val[0] + 'm';

lib/system/has-flag.js

@@ -3,21 +3,33 @@ MIT License
 
 Copyright (c) Sindre Sorhus <sindresorhus@gmail.com> (sindresorhus.com)
 
-Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:
+Permission is hereby granted, free of charge, to any person obtaining a copy of
+this software and associated documentation files (the "Software"), to deal in
+the Software without restriction, including without limitation the rights to
+use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies
+of the Software, and to permit persons to whom the Software is furnished to do
+so, subject to the following conditions:
 
-The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
 
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.
 */
 
 'use strict';
 
-module.exports = function (flag, argv) {
+module.exports = function(flag, argv) {
-	argv = argv || process.argv;
+  argv = argv || process.argv;
 
-	var terminatorPos = argv.indexOf('--');
+  var terminatorPos = argv.indexOf('--');
-	var prefix = /^-{1,2}/.test(flag) ? '' : '--';
+  var prefix = /^-{1,2}/.test(flag) ? '' : '--';
-	var pos = argv.indexOf(prefix + flag);
+  var pos = argv.indexOf(prefix + flag);
 
-	return pos !== -1 && (terminatorPos === -1 ? true : pos < terminatorPos);
+  return pos !== -1 && (terminatorPos === -1 ? true : pos < terminatorPos);
 };

lib/system/supports-colors.js

@@ -32,116 +32,120 @@ var env = process.env;
 
 var forceColor = void 0;
 if (hasFlag('no-color') || hasFlag('no-colors') || hasFlag('color=false')) {
-	forceColor = false;
+  forceColor = false;
-} else if (hasFlag('color') || hasFlag('colors') || hasFlag('color=true') || hasFlag('color=always')) {
+} else if (hasFlag('color') || hasFlag('colors') || hasFlag('color=true')
-	forceColor = true;
+           || hasFlag('color=always')) {
+  forceColor = true;
 }
 if ('FORCE_COLOR' in env) {
-	forceColor = env.FORCE_COLOR.length === 0 || parseInt(env.FORCE_COLOR, 10) !== 0;
+  forceColor = env.FORCE_COLOR.length === 0
+    || parseInt(env.FORCE_COLOR, 10) !== 0;
 }
 
 function translateLevel(level) {
-	if (level === 0) {
+  if (level === 0) {
-		return false;
+    return false;
-	}
+  }
 
-	return {
+  return {
-		level: level,
+    level: level,
-		hasBasic: true,
+    hasBasic: true,
-		has256: level >= 2,
+    has256: level >= 2,
-		has16m: level >= 3
+    has16m: level >= 3,
-	};
+  };
 }
 
 function supportsColor(stream) {
-	if (forceColor === false) {
+  if (forceColor === false) {
-		return 0;
+    return 0;
-	}
+  }
 
-	if (hasFlag('color=16m') || hasFlag('color=full') || hasFlag('color=truecolor')) {
+  if (hasFlag('color=16m') || hasFlag('color=full')
-		return 3;
+      || hasFlag('color=truecolor')) {
-	}
+    return 3;
+  }
 
-	if (hasFlag('color=256')) {
+  if (hasFlag('color=256')) {
-		return 2;
+    return 2;
-	}
+  }
 
-	if (stream && !stream.isTTY && forceColor !== true) {
+  if (stream && !stream.isTTY && forceColor !== true) {
-		return 0;
+    return 0;
-	}
+  }
 
-	var min = forceColor ? 1 : 0;
+  var min = forceColor ? 1 : 0;
 
-	if (process.platform === 'win32') {
+  if (process.platform === 'win32') {
-		// Node.js 7.5.0 is the first version of Node.js to include a patch to
+    // Node.js 7.5.0 is the first version of Node.js to include a patch to
-		// libuv that enables 256 color output on Windows. Anything earlier and it
+    // libuv that enables 256 color output on Windows. Anything earlier and it
-		// won't work. However, here we target Node.js 8 at minimum as it is an LTS
+    // won't work. However, here we target Node.js 8 at minimum as it is an LTS
-		// release, and Node.js 7 is not. Windows 10 build 10586 is the first Windows
+    // release, and Node.js 7 is not. Windows 10 build 10586 is the first
-		// release that supports 256 colors. Windows 10 build 14931 is the first release
+    // Windows release that supports 256 colors. Windows 10 build 14931 is the
-		// that supports 16m/TrueColor.
+    // first release that supports 16m/TrueColor.
-		var osRelease = os.release().split('.');
+    var osRelease = os.release().split('.');
-		if (Number(process.versions.node.split('.')[0]) >= 8 && Number(osRelease[0]) >= 10 && Number(osRelease[2]) >= 10586) {
+    if (Number(process.versions.node.split('.')[0]) >= 8
-			return Number(osRelease[2]) >= 14931 ? 3 : 2;
+        && Number(osRelease[0]) >= 10 && Number(osRelease[2]) >= 10586) {
-		}
+      return Number(osRelease[2]) >= 14931 ? 3 : 2;
+    }
 
-		return 1;
+    return 1;
-	}
+  }
 
-	if ('CI' in env) {
+  if ('CI' in env) {
-		if (['TRAVIS', 'CIRCLECI', 'APPVEYOR', 'GITLAB_CI'].some(function (sign) {
+    if (['TRAVIS', 'CIRCLECI', 'APPVEYOR', 'GITLAB_CI'].some(function(sign) {
-			return sign in env;
+      return sign in env;
-		}) || env.CI_NAME === 'codeship') {
+    }) || env.CI_NAME === 'codeship') {
-			return 1;
+      return 1;
-		}
+    }
 
-		return min;
+    return min;
-	}
+  }
 
-	if ('TEAMCITY_VERSION' in env) {
+  if ('TEAMCITY_VERSION' in env) {
-		return (/^(9\.(0*[1-9]\d*)\.|\d{2,}\.)/.test(env.TEAMCITY_VERSION) ? 1 : 0
+    return (/^(9\.(0*[1-9]\d*)\.|\d{2,}\.)/.test(env.TEAMCITY_VERSION) ? 1 : 0
-		);
+    );
-	}
+  }
 
-	if ('TERM_PROGRAM' in env) {
+  if ('TERM_PROGRAM' in env) {
-		var version = parseInt((env.TERM_PROGRAM_VERSION || '').split('.')[0], 10);
+    var version = parseInt((env.TERM_PROGRAM_VERSION || '').split('.')[0], 10);
 
-		switch (env.TERM_PROGRAM) {
+    switch (env.TERM_PROGRAM) {
-			case 'iTerm.app':
+      case 'iTerm.app':
-				return version >= 3 ? 3 : 2;
+        return version >= 3 ? 3 : 2;
-			case 'Hyper':
+      case 'Hyper':
-				return 3;
+        return 3;
-			case 'Apple_Terminal':
+      case 'Apple_Terminal':
-				return 2;
+        return 2;
-			// No default
+      // No default
-		}
+    }
-	}
+  }
 
-	if (/-256(color)?$/i.test(env.TERM)) {
+  if (/-256(color)?$/i.test(env.TERM)) {
-		return 2;
+    return 2;
-	}
+  }
 
-	if (/^screen|^xterm|^vt100|^rxvt|color|ansi|cygwin|linux/i.test(env.TERM)) {
+  if (/^screen|^xterm|^vt100|^rxvt|color|ansi|cygwin|linux/i.test(env.TERM)) {
-		return 1;
+    return 1;
-	}
+  }
 
-	if ('COLORTERM' in env) {
+  if ('COLORTERM' in env) {
-		return 1;
+    return 1;
-	}
+  }
 
-	if (env.TERM === 'dumb') {
+  if (env.TERM === 'dumb') {
-		return min;
+    return min;
-	}
+  }
 
-	return min;
+  return min;
 }
 
 function getSupportLevel(stream) {
-	var level = supportsColor(stream);
+  var level = supportsColor(stream);
-	return translateLevel(level);
+  return translateLevel(level);
 }
 
 module.exports = {
-	supportsColor: getSupportLevel,
+  supportsColor: getSupportLevel,
-	stdout: getSupportLevel(process.stdout),
+  stdout: getSupportLevel(process.stdout),
-	stderr: getSupportLevel(process.stderr)
+  stderr: getSupportLevel(process.stderr),
 };

package.json

@@ -1,18 +1,23 @@
 {
     "name": "colors",
     "description": "get colors in your node.js console",
-    "version": "1.2.2",
+    "version": "1.2.3",
     "author": "Marak Squires",
     "homepage": "https://github.com/Marak/colors.js",
     "bugs": "https://github.com/Marak/colors.js/issues",
-    "keywords": [ "ansi", "terminal", "colors" ],
+    "keywords": [
+        "ansi",
+        "terminal",
+        "colors"
+    ],
     "repository": {
         "type": "git",
         "url": "http://github.com/Marak/colors.js.git"
     },
     "license": "MIT",
     "scripts": {
-      "test": "node tests/basic-test.js && node tests/safe-test.js"
+        "lint": "eslint . --fix",
+        "test": "node tests/basic-test.js && node tests/safe-test.js"
     },
     "engines": {
         "node": ">=0.1.90"
@@ -26,5 +31,9 @@
         "themes",
         "index.d.ts",
         "safe.d.ts"
-    ]
+    ],
+    "devDependencies": {
+        "eslint": "^4.19.1",
+        "eslint-config-google": "^0.9.1"
+    }
 }

safe.js

@@ -1,5 +1,6 @@
 //
-// Remark: Requiring this file will use the "safe" colors API which will not touch String.prototype
+// Remark: Requiring this file will use the "safe" colors API,
+// which will not touch String.prototype.
 //
 //   var colors = require('colors/safe');
 //   colors.red("foo")

tests/basic-test.js

@@ -1,5 +1,5 @@
-var assert = require('assert'),
+var assert = require('assert');
-    colors = require('../lib/index');
+var colors = require('../lib/index');
 
 var s = 'string';
 
@@ -15,12 +15,11 @@ function aE(s, color, code) {
   assert.equal(s[color].strip, colors.strip(s));
 }
 
-function h(s, color) {
+var stylesColors = ['white', 'black', 'blue', 'cyan', 'green', 'magenta',
-  return '<span style="color:' + color + ';">' + s + '</span>';
+  'red', 'yellow'];
-}
+// eslint-disable-next-line
-
+var stylesAll = stylesColors.concat(['bold', 'italic', 'underline',
-var stylesColors = ['white', 'black', 'blue', 'cyan', 'green', 'magenta', 'red', 'yellow'];
+  'inverse', 'rainbow']);
-var stylesAll = stylesColors.concat(['bold', 'italic', 'underline', 'inverse', 'rainbow']);
 
 colors.mode = 'console';
 assert.equal(s.bold, '\x1B[1m' + s + '\x1B[22m');
@@ -46,16 +45,19 @@ assert.equal(s, 'string');
 var testStringWithNewLines = s + '\n' + s;
 
 // single style
-assert.equal(testStringWithNewLines.red, '\x1b[31m' + s + '\n' + s + '\x1b[39m');
+assert.equal(testStringWithNewLines.red, '\x1b[31m' + s + '\n' + s +
+  '\x1b[39m');
 
 var testStringWithNewLinesStyled = s.underline + '\n' + s.bold;
 
 // nested styles
-assert.equal(testStringWithNewLinesStyled.red, '\x1b[31m' + '\x1b[4m' + s + '\x1b[24m' + '\n' + '\x1b[1m' + s + '\x1b[22m' + '\x1b[39m');
+assert.equal(testStringWithNewLinesStyled.red,
+  '\x1b[31m' + '\x1b[4m' + s + '\x1b[24m' + '\n' + '\x1b[1m' + s +
+  '\x1b[22m' + '\x1b[39m');
 
-colors.setTheme({ error: 'red' });
+colors.setTheme({error: 'red'});
 
-assert.equal(typeof ("astring".red), 'string');
+assert.equal(typeof ('astring'.red), 'string');
-assert.equal(typeof ("astring".error), 'string');
+assert.equal(typeof ('astring'.error), 'string');
 
 assert.equal(s, 'string');

tests/safe-test.js

@@ -1,5 +1,5 @@
-var assert = require('assert'),
+var assert = require('assert');
-    colors = require('../safe');
+var colors = require('../safe');
 
 var s = 'string';
 
@@ -12,12 +12,11 @@ function aE(s, color, code) {
   assert.equal(colors.strip(s), s);
 }
 
-function h(s, color) {
+var stylesColors = ['white', 'black', 'blue', 'cyan', 'green', 'magenta',
-  return '<span style="color:' + color + ';">' + s + '</span>';
+  'red', 'yellow'];
-}
+// eslint-disable-next-line
-
+var stylesAll = stylesColors.concat(['bold', 'italic', 'underline', 'inverse',
-var stylesColors = ['white', 'black', 'blue', 'cyan', 'green', 'magenta', 'red', 'yellow'];
+  'rainbow']);
-var stylesAll = stylesColors.concat(['bold', 'italic', 'underline', 'inverse', 'rainbow']);
 
 colors.mode = 'console';
 assert.equal(colors.bold(s), '\x1B[1m' + s + '\x1B[22m');
@@ -43,15 +42,18 @@ assert.equal(s, 'string');
 var testStringWithNewLines = s + '\n' + s;
 
 // single style
-assert.equal(colors.red(testStringWithNewLines), '\x1b[31m' + s + '\x1b[39m' + '\n' + '\x1b[31m' + s + '\x1b[39m');
+assert.equal(colors.red(testStringWithNewLines),
+  '\x1b[31m' + s + '\x1b[39m' + '\n' + '\x1b[31m' + s + '\x1b[39m');
 
 var testStringWithNewLinesStyled = colors.underline(s) + '\n' + colors.bold(s);
 
 // nested styles
-assert.equal(colors.red(testStringWithNewLinesStyled), '\x1b[31m' + '\x1b[4m' + s + '\x1b[24m' + '\x1b[39m' + '\n' + '\x1b[31m' + '\x1b[1m' + s + '\x1b[22m' + '\x1b[39m');
+assert.equal(colors.red(testStringWithNewLinesStyled),
+  '\x1b[31m' + '\x1b[4m' + s + '\x1b[24m' + '\x1b[39m' + '\n' + '\x1b[31m' +
+  '\x1b[1m' + s + '\x1b[22m' + '\x1b[39m');
 
-colors.setTheme({ error: 'red' });
+colors.setTheme({error: 'red'});
 
-assert.equal(typeof (colors.red("astring")), 'string');
+assert.equal(typeof (colors.red('astring')), 'string');
-assert.equal(typeof (colors.error("astring")), 'string');
+assert.equal(typeof (colors.error('astring')), 'string');
 

themes/generic-logging.js

@@ -8,5 +8,5 @@ module['exports'] = {
   help: 'cyan',
   warn: 'yellow',
   debug: 'blue',
-  error: 'red'
+  error: 'red',
 };