coffeelint 1.9.1 → 1.10.0.pre.patch.1

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 7720f637d2867a5667ef76f393504a66af63150f
+  data.tar.gz: fccb84fdb0d07e5da9d45787eafca17e88fd533a
+SHA512:
+  metadata.gz: e1132114822ce18e09c6661b557fde784ffeeb3150a72dd568eb147dddd456d318e8e0b2508eff81ed2f7a5fca1647e28b209e1871dc70590c5b8b5e8ae9dd25
+  data.tar.gz: 0c6fcb815fca593829c6e05d0edb0f2afdec674299f037b188e8947450ca3ce46700d0a7f4c3a0c644d55eac5274b897b07a4042572af5eadd5253a2b3f75901

data/README.md

@@ -1,6 +1,6 @@
 # Coffeelint [![Build Status](https://travis-ci.org/zmbush/coffeelint-ruby.svg?branch=master)](https://travis-ci.org/zmbush/coffeelint-ruby) [![Gem Version](https://badge.fury.io/rb/coffeelint.png)](http://badge.fury.io/rb/coffeelint)
 
-Using coffeelint version: v1.9.1
+Using coffeelint version: v1.10.0
 
 Coffeelint is a set of simple ruby bindings for [coffeelint](https://github.com/clutchski/coffeelint).
 
@@ -36,7 +36,7 @@ Coffeelint.lint_dir(directory, [config_options]) do |filename, lint_report|
     Coffeelint.display_test_results(filename, lint_report)
 end
 Coffeelint.run_test(filename of coffeescript source, [config_options]) # Run tests and print pretty results (return true/false)
-Coffeelint.run_test_suite(directory, [config_options]) # Runs a pretty report recursively for a directory (return true/false)
+Coffeelint.run_test_suite(directory, [config_options]) # Runs a pretty report recursively for a directory (returns/exits with number of errors if any or 0)
 ```
 
 ### Config Options

data/coffeelint/lib/coffeelint.js

@@ -6,7 +6,7 @@ CoffeeLint
 Copyright (c) 2011 Matthew Perpick.
 CoffeeLint is freely distributable under the MIT license.
  */
-var ASTLinter, CoffeeScript, ERROR, ErrorReport, IGNORE, LexicalLinter, LineLinter, RULES, WARN, _rules, cache, coffeelint, defaults, difference, extend, hasSyntaxError, mergeDefaultConfig, nodeRequire, packageJSON,
+var ASTLinter, CoffeeScript, ERROR, ErrorReport, IGNORE, LexicalLinter, LineLinter, RULES, WARN, _rules, cache, coffeelint, defaults, difference, extend, hasSyntaxError, mergeDefaultConfig, nodeRequire, packageJSON, sameJSON,
   slice = [].slice,
   indexOf = [].indexOf || function(item) { for (var i = 0, l = this.length; i < l; i++) { if (i in this && this[i] === item) return i; } return -1; };
 
@@ -78,8 +78,15 @@ ASTLinter = require('./ast_linter.coffee');
 cache = null;
 
 mergeDefaultConfig = function(userConfig) {
-  var config, rule, ruleConfig;
+  var config, rule, ruleConfig, ruleLoader;
+  try {
+    ruleLoader = nodeRequire('./ruleLoader');
+    ruleLoader.loadFromConfig(coffeelint, userConfig);
+  } catch (_error) {}
   config = {};
+  if (userConfig.coffeelint) {
+    config.coffeelint = userConfig.coffeelint;
+  }
   for (rule in RULES) {
     ruleConfig = RULES[rule];
     config[rule] = defaults(userConfig[rule], ruleConfig);
@@ -87,6 +94,50 @@ mergeDefaultConfig = function(userConfig) {
   return config;
 };
 
+sameJSON = function(a, b) {
+  return JSON.stringify(a) === JSON.stringify(b);
+};
+
+coffeelint.trimConfig = function(userConfig) {
+  var config, dConfig, dValue, key, newConfig, ref, rule, value;
+  newConfig = {};
+  userConfig = mergeDefaultConfig(userConfig);
+  for (rule in userConfig) {
+    config = userConfig[rule];
+    dConfig = RULES[rule];
+    if (rule === 'coffeelint') {
+      config.transforms = config._transforms;
+      delete config._transforms;
+      config.coffeescript = config._coffeescript;
+      delete config._coffeescript;
+      newConfig[rule] = config;
+    } else if ((config.level === (ref = dConfig.level) && ref === 'ignore')) {
+      void 0;
+    } else if (config.level === 'ignore') {
+      newConfig[rule] = {
+        level: 'ignore'
+      };
+    } else {
+      config.module = config._module;
+      delete config._module;
+      for (key in config) {
+        value = config[key];
+        if (key === 'message' || key === 'description' || key === 'name') {
+          continue;
+        }
+        dValue = dConfig[key];
+        if (value !== dValue && !sameJSON(value, dValue)) {
+          if (newConfig[rule] == null) {
+            newConfig[rule] = {};
+          }
+          newConfig[rule][key] = value;
+        }
+      }
+    }
+  }
+  return newConfig;
+};
+
 coffeelint.invertLiterate = function(source) {
   var len, line, n, newSource, ref;
   source = CoffeeScript.helpers.invertLiterate(source);
@@ -221,6 +272,10 @@ coffeelint.registerRule(require('./rules/transform_messes_up_line_numbers.coffee
 
 coffeelint.registerRule(require('./rules/ensure_comprehensions.coffee'));
 
+coffeelint.registerRule(require('./rules/no_this.coffee'));
+
+coffeelint.registerRule(require('./rules/eol_last.coffee'));
+
 hasSyntaxError = function(source) {
   try {
     CoffeeScript.tokens(source);
@@ -244,10 +299,6 @@ coffeelint.lint = function(source, userConfig, literate) {
     literate = false;
   }
   errors = [];
-  try {
-    ruleLoader = nodeRequire('./ruleLoader');
-    ruleLoader.loadFromConfig(this, userConfig);
-  } catch (_error) {}
   if (cache != null) {
     cache.setConfig(userConfig);
   }
@@ -263,15 +314,17 @@ coffeelint.lint = function(source, userConfig, literate) {
     ref2 = userConfig != null ? (ref1 = userConfig.coffeelint) != null ? ref1.transforms : void 0 : void 0;
     for (n = 0, len = ref2.length; n < len; n++) {
       m = ref2[n];
-      transform = ruleLoader.require(m);
-      source = transform(source);
+      try {
+        ruleLoader = nodeRequire('./ruleLoader');
+        transform = ruleLoader.require(m);
+        source = transform(source);
+      } catch (_error) {}
     }
     if (sourceLength !== source.split("\n").length && config.transform_messes_up_line_numbers.level !== 'ignore') {
       errors.push(extend({
         lineNumber: 1,
         context: "File was transformed from " + sourceLength + " lines to " + (source.split("\n").length) + " lines"
       }, config.transform_messes_up_line_numbers));
-      console.log(errors);
     }
   }
   if ((userConfig != null ? (ref3 = userConfig.coffeelint) != null ? ref3.coffeescript : void 0 : void 0) != null) {
@@ -368,11 +421,11 @@ coffeelint.setCache = function(obj) {
 
 
 
-},{"./../package.json":2,"./ast_linter.coffee":3,"./error_report.coffee":5,"./lexical_linter.coffee":6,"./line_linter.coffee":7,"./rules.coffee":8,"./rules/arrow_spacing.coffee":9,"./rules/braces_spacing.coffee":10,"./rules/camel_case_classes.coffee":11,"./rules/colon_assignment_spacing.coffee":12,"./rules/cyclomatic_complexity.coffee":13,"./rules/duplicate_key.coffee":14,"./rules/empty_constructor_needs_parens.coffee":15,"./rules/ensure_comprehensions.coffee":16,"./rules/indentation.coffee":17,"./rules/line_endings.coffee":18,"./rules/max_line_length.coffee":19,"./rules/missing_fat_arrows.coffee":20,"./rules/newlines_after_classes.coffee":21,"./rules/no_backticks.coffee":22,"./rules/no_debugger.coffee":23,"./rules/no_empty_functions.coffee":24,"./rules/no_empty_param_list.coffee":25,"./rules/no_implicit_braces.coffee":26,"./rules/no_implicit_parens.coffee":27,"./rules/no_interpolation_in_single_quotes.coffee":28,"./rules/no_plusplus.coffee":29,"./rules/no_stand_alone_at.coffee":30,"./rules/no_tabs.coffee":31,"./rules/no_throwing_strings.coffee":32,"./rules/no_trailing_semicolons.coffee":33,"./rules/no_trailing_whitespace.coffee":34,"./rules/no_unnecessary_double_quotes.coffee":35,"./rules/no_unnecessary_fat_arrows.coffee":36,"./rules/non_empty_constructor_needs_parens.coffee":37,"./rules/prefer_english_operator.coffee":38,"./rules/space_operators.coffee":39,"./rules/spacing_after_comma.coffee":40,"./rules/transform_messes_up_line_numbers.coffee":41}],2:[function(require,module,exports){
+},{"./../package.json":2,"./ast_linter.coffee":3,"./error_report.coffee":5,"./lexical_linter.coffee":6,"./line_linter.coffee":7,"./rules.coffee":8,"./rules/arrow_spacing.coffee":9,"./rules/braces_spacing.coffee":10,"./rules/camel_case_classes.coffee":11,"./rules/colon_assignment_spacing.coffee":12,"./rules/cyclomatic_complexity.coffee":13,"./rules/duplicate_key.coffee":14,"./rules/empty_constructor_needs_parens.coffee":15,"./rules/ensure_comprehensions.coffee":16,"./rules/eol_last.coffee":17,"./rules/indentation.coffee":18,"./rules/line_endings.coffee":19,"./rules/max_line_length.coffee":20,"./rules/missing_fat_arrows.coffee":21,"./rules/newlines_after_classes.coffee":22,"./rules/no_backticks.coffee":23,"./rules/no_debugger.coffee":24,"./rules/no_empty_functions.coffee":25,"./rules/no_empty_param_list.coffee":26,"./rules/no_implicit_braces.coffee":27,"./rules/no_implicit_parens.coffee":28,"./rules/no_interpolation_in_single_quotes.coffee":29,"./rules/no_plusplus.coffee":30,"./rules/no_stand_alone_at.coffee":31,"./rules/no_tabs.coffee":32,"./rules/no_this.coffee":33,"./rules/no_throwing_strings.coffee":34,"./rules/no_trailing_semicolons.coffee":35,"./rules/no_trailing_whitespace.coffee":36,"./rules/no_unnecessary_double_quotes.coffee":37,"./rules/no_unnecessary_fat_arrows.coffee":38,"./rules/non_empty_constructor_needs_parens.coffee":39,"./rules/prefer_english_operator.coffee":40,"./rules/space_operators.coffee":41,"./rules/spacing_after_comma.coffee":42,"./rules/transform_messes_up_line_numbers.coffee":43}],2:[function(require,module,exports){
 module.exports={
   "name": "coffeelint",
   "description": "Lint your CoffeeScript",
-  "version": "1.9.1",
+  "version": "1.10.0",
   "homepage": "http://www.coffeelint.org",
   "keywords": [
     "lint",
@@ -399,21 +452,18 @@ module.exports={
     "glob": "^4.0.0",
     "ignore": "^2.2.15",
     "optimist": "^0.6.1",
-    "resolve": "^0.6.3"
+    "resolve": "^0.6.3",
+    "strip-json-comments": "^1.0.2"
   },
   "devDependencies": {
     "vows": ">=0.6.0",
     "underscore": ">=1.4.4"
   },
-  "licenses": [
-    {
-      "type": "MIT",
-      "url": "http://github.com/clutchski/coffeelint/raw/master/LICENSE"
-    }
-  ],
+  "license": "MIT",
   "scripts": {
     "pretest": "cake compile",
     "test": "./vowsrunner.js --spec test/*.coffee test/*.litcoffee",
+    "testrule": "npm run compile && ./vowsrunner.js --spec",
     "posttest": "npm run lint",
     "prepublish": "cake prepublish",
     "publish": "cake publish",
@@ -1053,11 +1103,11 @@ module.exports = ArrowSpacing = (function() {
   ArrowSpacing.prototype.rule = {
     name: 'arrow_spacing',
     level: 'ignore',
-    message: 'Function arrow (->) must be spaced properly',
+    message: 'Function arrows (-> and =>) must be spaced properly',
     description: "<p>This rule checks to see that there is spacing before and after\nthe arrow operator that declares a function. This rule is disabled\nby default.</p> <p>Note that if arrow_spacing is enabled, and you\npass an empty function as a parameter, arrow_spacing will accept\neither a space or no space in-between the arrow operator and the\nparenthesis</p>\n<pre><code># Both of this will not trigger an error,\n# even with arrow_spacing enabled.\nx(-> 3)\nx( -> 3)\n\n# However, this will trigger an error\nx((a,b)-> 3)\n</code>\n</pre>"
   };
 
-  ArrowSpacing.prototype.tokens = ['->'];
+  ArrowSpacing.prototype.tokens = ['->', '=>'];
 
   ArrowSpacing.prototype.lintToken = function(token, tokenApi) {
     var pp;
@@ -1100,8 +1150,9 @@ module.exports = BracesSpacing = (function() {
     name: 'braces_spacing',
     level: 'ignore',
     spaces: 0,
+    empty_object_spaces: 0,
     message: 'Curly braces must have the proper spacing',
-    description: 'This rule checks to see that there is the proper spacing inside\ncurly braces. The spacing amount is specified by "spaces".\n\n<pre><code>\n# Spaces is 0\n{a: b}     # Good\n{a: b }    # Bad\n{ a: b}    # Bad\n{ a: b }   # Bad\n\n# Spaces is 1\n{a: b}     # Bad\n{a: b }    # Bad\n{ a: b}    # Bad\n{ a: b }   # Good\n{ a: b  }  # Bad\n{  a: b }  # Bad\n{  a: b  } # Bad\n</code></pre>\n\nThis rule is disabled by default.'
+    description: 'This rule checks to see that there is the proper spacing inside\ncurly braces. The spacing amount is specified by "spaces".\nThe spacing amount for empty objects is specified by\n"empty_object_spaces".\n\n<pre><code>\n# Spaces is 0\n{a: b}     # Good\n{a: b }    # Bad\n{ a: b}    # Bad\n{ a: b }   # Bad\n\n# Spaces is 1\n{a: b}     # Bad\n{a: b }    # Bad\n{ a: b}    # Bad\n{ a: b }   # Good\n{ a: b  }  # Bad\n{  a: b }  # Bad\n{  a: b  } # Bad\n\n# Empty Object Spaces is 0\n{}         # Good\n{ }        # Bad\n\n# Empty Object Spaces is 1\n{}         # Bad\n{ }        # Good\n</code></pre>\n\nThis rule is disabled by default.'
   };
 
   BracesSpacing.prototype.tokens = ['{', '}'];
@@ -1127,6 +1178,16 @@ module.exports = BracesSpacing = (function() {
     return firstToken[2].first_line === secondToken[2].first_line;
   };
 
+  BracesSpacing.prototype.getExpectedSpaces = function(tokenApi, firstToken, secondToken) {
+    var config, ref;
+    config = tokenApi.config[this.rule.name];
+    if (firstToken[0] === '{' && secondToken[0] === '}') {
+      return (ref = config.empty_object_spaces) != null ? ref : config.spaces;
+    } else {
+      return config.spaces;
+    }
+  };
+
   BracesSpacing.prototype.lintToken = function(token, tokenApi) {
     var actual, expected, firstToken, msg, ref, secondToken;
     if (token.generated) {
@@ -1136,7 +1197,7 @@ module.exports = BracesSpacing = (function() {
     if (!this.tokensOnSameLine(firstToken, secondToken)) {
       return null;
     }
-    expected = tokenApi.config[this.rule.name].spaces;
+    expected = this.getExpectedSpaces(tokenApi, firstToken, secondToken);
     actual = this.distanceBetweenTokens(firstToken, secondToken);
     if (actual === expected) {
       return null;
@@ -1162,7 +1223,7 @@ module.exports = BracesSpacing = (function() {
 var CamelCaseClasses, regexes;
 
 regexes = {
-  camelCase: /^[A-Z][a-zA-Z\d]*$/
+  camelCase: /^[A-Z_][a-zA-Z\d]*$/
 };
 
 module.exports = CamelCaseClasses = (function() {
@@ -1171,8 +1232,8 @@ module.exports = CamelCaseClasses = (function() {
   CamelCaseClasses.prototype.rule = {
     name: 'camel_case_classes',
     level: 'error',
-    message: 'Class names should be camel cased',
-    description: "This rule mandates that all class names are CamelCased. Camel\ncasing class names is a generally accepted way of distinguishing\nconstructor functions - which require the 'new' prefix to behave\nproperly - from plain old functions.\n<pre>\n<code># Good!\nclass BoaConstrictor\n\n# Bad!\nclass boaConstrictor\n</code>\n</pre>\nThis rule is enabled by default."
+    message: 'Class name should be UpperCamelCased',
+    description: "This rule mandates that all class names are UpperCamelCased.\nCamel casing class names is a generally accepted way of\ndistinguishing constructor functions - which require the 'new'\nprefix to behave properly - from plain old functions.\n<pre>\n<code># Good!\nclass BoaConstrictor\n\n# Bad!\nclass boaConstrictor\n</code>\n</pre>\nThis rule is enabled by default."
   };
 
   CamelCaseClasses.prototype.tokens = ['CLASS'];
@@ -1407,17 +1468,24 @@ module.exports = EmptyConstructorNeedsParens = (function() {
   EmptyConstructorNeedsParens.prototype.tokens = ['UNARY'];
 
   EmptyConstructorNeedsParens.prototype.lintToken = function(token, tokenApi) {
-    var expectedCallStart, expectedIdentifier, identifierIndex;
+    var expectedCallStart, expectedIdentifier, identifierIndex, peek, ref;
     if (token[1] === 'new') {
+      peek = tokenApi.peek.bind(tokenApi);
       identifierIndex = 1;
       while (true) {
-        expectedIdentifier = tokenApi.peek(identifierIndex);
-        expectedCallStart = tokenApi.peek(identifierIndex + 1);
+        expectedIdentifier = peek(identifierIndex);
+        expectedCallStart = peek(identifierIndex + 1);
         if ((expectedIdentifier != null ? expectedIdentifier[0] : void 0) === 'IDENTIFIER') {
           if ((expectedCallStart != null ? expectedCallStart[0] : void 0) === '.') {
             identifierIndex += 2;
             continue;
           }
+          if ((expectedCallStart != null ? expectedCallStart[0] : void 0) === 'INDEX_START') {
+            while (((ref = peek(identifierIndex)) != null ? ref[0] : void 0) !== 'INDEX_END') {
+              identifierIndex++;
+            }
+            continue;
+          }
         }
         break;
       }
@@ -1507,6 +1575,34 @@ module.exports = EnsureComprehensions = (function() {
 
 
 },{}],17:[function(require,module,exports){
+var EOLLast;
+
+module.exports = EOLLast = (function() {
+  function EOLLast() {}
+
+  EOLLast.prototype.rule = {
+    name: 'eol_last',
+    level: 'ignore',
+    message: 'File does not end with a single newline',
+    description: "Checks that the file ends with a single newline"
+  };
+
+  EOLLast.prototype.lintLine = function(line, lineApi) {
+    if (!lineApi.isLastLine()) {
+      return null;
+    }
+    if (line.length) {
+      return true;
+    }
+  };
+
+  return EOLLast;
+
+})();
+
+
+
+},{}],18:[function(require,module,exports){
 var Indentation;
 
 module.exports = Indentation = (function() {
@@ -1631,7 +1727,7 @@ module.exports = Indentation = (function() {
 
 
 
-},{}],18:[function(require,module,exports){
+},{}],19:[function(require,module,exports){
 var LineEndings;
 
 module.exports = LineEndings = (function() {
@@ -1676,7 +1772,7 @@ module.exports = LineEndings = (function() {
 
 
 
-},{}],19:[function(require,module,exports){
+},{}],20:[function(require,module,exports){
 var MaxLineLength, regexes;
 
 regexes = {
@@ -1700,7 +1796,7 @@ module.exports = MaxLineLength = (function() {
     var limitComments, lineLength, max, ref, ref1;
     max = (ref = lineApi.config[this.rule.name]) != null ? ref.value : void 0;
     limitComments = (ref1 = lineApi.config[this.rule.name]) != null ? ref1.limitComments : void 0;
-    lineLength = line.trimRight().length;
+    lineLength = line.replace(/\s+$/, '').length;
     if (lineApi.isLiterate() && regexes.literateComment.test(line)) {
       lineLength -= 2;
     }
@@ -1722,7 +1818,7 @@ module.exports = MaxLineLength = (function() {
 
 
 
-},{}],20:[function(require,module,exports){
+},{}],21:[function(require,module,exports){
 var MissingFatArrows, any, containsButIsnt,
   bind = function(fn, me){ return function(){ return fn.apply(me, arguments); }; },
   indexOf = [].indexOf || function(item) { for (var i = 0, l = this.length; i < l; i++) { if (i in this && this[i] === item) return i; } return -1; };
@@ -1761,8 +1857,9 @@ module.exports = MissingFatArrows = (function() {
   MissingFatArrows.prototype.rule = {
     name: 'missing_fat_arrows',
     level: 'ignore',
+    is_strict: false,
     message: 'Used `this` in a function without a fat arrow',
-    description: "Warns when you use `this` inside a function that wasn't defined\nwith a fat arrow. This rule does not apply to methods defined in a\nclass, since they have `this` bound to the class instance (or the\nclass itself, for class methods).\n\nIt is impossible to statically determine whether a function using\n`this` will be bound with the correct `this` value due to language\nfeatures like `Function.prototype.call` and\n`Function.prototype.bind`, so this rule may produce false positives."
+    description: "Warns when you use `this` inside a function that wasn't defined\nwith a fat arrow. This rule does not apply to methods defined in a\nclass, since they have `this` bound to the class instance (or the\nclass itself, for class methods). The option `is_strict` is\navailable for checking bindings of class methods.\n\nIt is impossible to statically determine whether a function using\n`this` will be bound with the correct `this` value due to language\nfeatures like `Function.prototype.call` and\n`Function.prototype.bind`, so this rule may produce false positives."
   };
 
   MissingFatArrows.prototype.lintAST = function(node, astApi) {
@@ -1772,11 +1869,15 @@ module.exports = MissingFatArrows = (function() {
   };
 
   MissingFatArrows.prototype.lintNode = function(node, methods) {
-    var error;
+    var error, is_strict, ref;
     if (methods == null) {
       methods = [];
     }
-    if ((!this.isFatArrowCode(node)) && (indexOf.call(methods, node) < 0) && (this.needsFatArrow(node))) {
+    is_strict = (ref = this.astApi.config[this.rule.name]) != null ? ref.is_strict : void 0;
+    if (this.isConstructor(node)) {
+      return;
+    }
+    if ((!this.isFatArrowCode(node)) && (is_strict ? true : indexOf.call(methods, node) < 0) && (this.needsFatArrow(node))) {
       error = this.astApi.createError({
         lineNumber: node.locationData.first_line + 1
       });
@@ -1822,6 +1923,11 @@ module.exports = MissingFatArrows = (function() {
     return this.isCode(node) && node.bound;
   };
 
+  MissingFatArrows.prototype.isConstructor = function(node) {
+    var ref, ref1;
+    return ((ref = node.variable) != null ? (ref1 = ref.base) != null ? ref1.value : void 0 : void 0) === 'constructor';
+  };
+
   MissingFatArrows.prototype.needsFatArrow = function(node) {
     return this.isCode(node) && (any(node.params, (function(_this) {
       return function(param) {
@@ -1849,7 +1955,7 @@ module.exports = MissingFatArrows = (function() {
 
 
 
-},{}],21:[function(require,module,exports){
+},{}],22:[function(require,module,exports){
 var NewlinesAfterClasses;
 
 module.exports = NewlinesAfterClasses = (function() {
@@ -1860,7 +1966,7 @@ module.exports = NewlinesAfterClasses = (function() {
     value: 3,
     level: 'ignore',
     message: 'Wrong count of newlines between a class and other code',
-    description: "Checks the number of newlines between classes and other code"
+    description: "<p>Checks the number of newlines between classes and other code.</p>\n\nOptions:\n- <pre><code>value</code></pre> - The number of required newlines\nafter class definitions. Defaults to 3."
   };
 
   NewlinesAfterClasses.prototype.lintLine = function(line, lineApi) {
@@ -1885,7 +1991,7 @@ module.exports = NewlinesAfterClasses = (function() {
 
 
 
-},{}],22:[function(require,module,exports){
+},{}],23:[function(require,module,exports){
 var NoBackticks;
 
 module.exports = NoBackticks = (function() {
@@ -1910,7 +2016,7 @@ module.exports = NoBackticks = (function() {
 
 
 
-},{}],23:[function(require,module,exports){
+},{}],24:[function(require,module,exports){
 var NoDebugger;
 
 module.exports = NoDebugger = (function() {
@@ -1919,16 +2025,28 @@ module.exports = NoDebugger = (function() {
   NoDebugger.prototype.rule = {
     name: 'no_debugger',
     level: 'warn',
-    message: 'Debugger statements will cause warnings',
-    description: "This rule detects the `debugger` statement.\nThis rule is `warn` by default."
+    message: 'Found debugging code',
+    console: false,
+    description: "This rule detects `debugger` and optionally `console` calls\nThis rule is `warn` by default."
   };
 
-  NoDebugger.prototype.tokens = ["DEBUGGER"];
+  NoDebugger.prototype.tokens = ["DEBUGGER", "IDENTIFIER"];
 
   NoDebugger.prototype.lintToken = function(token, tokenApi) {
-    return {
-      context: "found '" + token[0] + "'"
-    };
+    var method, ref, ref1;
+    if (token[0] === 'DEBUGGER') {
+      return {
+        context: "found '" + token[0] + "'"
+      };
+    }
+    if ((ref = tokenApi.config[this.rule.name]) != null ? ref.console : void 0) {
+      if (token[1] === 'console' && ((ref1 = tokenApi.peek(1)) != null ? ref1[0] : void 0) === '.') {
+        method = tokenApi.peek(2);
+        return {
+          context: "found 'console." + method[1] + "'"
+        };
+      }
+    }
   };
 
   return NoDebugger;
@@ -1937,7 +2055,7 @@ module.exports = NoDebugger = (function() {
 
 
 
-},{}],24:[function(require,module,exports){
+},{}],25:[function(require,module,exports){
 var NoEmptyFunctions, isEmptyCode;
 
 isEmptyCode = function(node, astApi) {
@@ -1982,7 +2100,7 @@ module.exports = NoEmptyFunctions = (function() {
 
 
 
-},{}],25:[function(require,module,exports){
+},{}],26:[function(require,module,exports){
 var NoEmptyParamList;
 
 module.exports = NoEmptyParamList = (function() {
@@ -2009,7 +2127,7 @@ module.exports = NoEmptyParamList = (function() {
 
 
 
-},{}],26:[function(require,module,exports){
+},{}],27:[function(require,module,exports){
 var NoImplicitBraces;
 
 module.exports = NoImplicitBraces = (function() {
@@ -2069,7 +2187,7 @@ module.exports = NoImplicitBraces = (function() {
 
 
 
-},{}],27:[function(require,module,exports){
+},{}],28:[function(require,module,exports){
 var NoImplicitParens;
 
 module.exports = NoImplicitParens = (function() {
@@ -2112,7 +2230,7 @@ module.exports = NoImplicitParens = (function() {
 
 
 
-},{}],28:[function(require,module,exports){
+},{}],29:[function(require,module,exports){
 var NoInterpolationInSingleQuotes;
 
 module.exports = NoInterpolationInSingleQuotes = (function() {
@@ -2130,7 +2248,7 @@ module.exports = NoInterpolationInSingleQuotes = (function() {
   NoInterpolationInSingleQuotes.prototype.lintToken = function(token, tokenApi) {
     var hasInterpolation, tokenValue;
     tokenValue = token[1];
-    hasInterpolation = tokenValue.match(/#\{[^}]+\}/);
+    hasInterpolation = tokenValue.match(/^\'.*#\{[^}]+\}.*\'$/);
     return hasInterpolation;
   };
 
@@ -2140,7 +2258,7 @@ module.exports = NoInterpolationInSingleQuotes = (function() {
 
 
 
-},{}],29:[function(require,module,exports){
+},{}],30:[function(require,module,exports){
 var NoPlusPlus;
 
 module.exports = NoPlusPlus = (function() {
@@ -2167,7 +2285,7 @@ module.exports = NoPlusPlus = (function() {
 
 
 
-},{}],30:[function(require,module,exports){
+},{}],31:[function(require,module,exports){
 var NoStandAloneAt;
 
 module.exports = NoStandAloneAt = (function() {
@@ -2204,7 +2322,7 @@ module.exports = NoStandAloneAt = (function() {
 
 
 
-},{}],31:[function(require,module,exports){
+},{}],32:[function(require,module,exports){
 var NoTabs, indentationRegex,
   indexOf = [].indexOf || function(item) { for (var i = 0, l = this.length; i < l; i++) { if (i in this && this[i] === item) return i; } return -1; };
 
@@ -2236,7 +2354,32 @@ module.exports = NoTabs = (function() {
 
 
 
-},{}],32:[function(require,module,exports){
+},{}],33:[function(require,module,exports){
+var NoThis;
+
+module.exports = NoThis = (function() {
+  function NoThis() {}
+
+  NoThis.prototype.rule = {
+    name: 'no_this',
+    description: 'This rule prohibits \'this\'.\nUse \'@\' instead.',
+    level: 'ignore',
+    message: "Don't use 'this', use '@' instead"
+  };
+
+  NoThis.prototype.tokens = ['THIS'];
+
+  NoThis.prototype.lintToken = function(token, tokenApi) {
+    return true;
+  };
+
+  return NoThis;
+
+})();
+
+
+
+},{}],34:[function(require,module,exports){
 var NoThrowingStrings;
 
 module.exports = NoThrowingStrings = (function() {
@@ -2264,7 +2407,7 @@ module.exports = NoThrowingStrings = (function() {
 
 
 
-},{}],33:[function(require,module,exports){
+},{}],35:[function(require,module,exports){
 var NoTrailingSemicolons, regexes,
   indexOf = [].indexOf || function(item) { for (var i = 0, l = this.length; i < l; i++) { if (i in this && this[i] === item) return i; } return -1; },
   slice = [].slice;
@@ -2317,7 +2460,7 @@ module.exports = NoTrailingSemicolons = (function() {
 
 
 
-},{}],34:[function(require,module,exports){
+},{}],36:[function(require,module,exports){
 var NoTrailingWhitespace, regexes;
 
 regexes = {
@@ -2381,7 +2524,7 @@ module.exports = NoTrailingWhitespace = (function() {
 
 
 
-},{}],35:[function(require,module,exports){
+},{}],37:[function(require,module,exports){
 var NoUnnecessaryDoubleQuotes;
 
 module.exports = NoUnnecessaryDoubleQuotes = (function() {
@@ -2400,7 +2543,7 @@ module.exports = NoUnnecessaryDoubleQuotes = (function() {
   NoUnnecessaryDoubleQuotes.prototype.tokens = ['STRING', 'STRING_START', 'STRING_END'];
 
   NoUnnecessaryDoubleQuotes.prototype.lintToken = function(token, tokenApi) {
-    var hasLegalConstructs, stringValue, tokenValue, type;
+    var hasLegalConstructs, ref, stringValue, tokenValue, type;
     type = token[0], tokenValue = token[1];
     if (type === 'STRING_START' || type === 'STRING_END') {
       return this.trackParens.apply(this, arguments);
@@ -2409,6 +2552,9 @@ module.exports = NoUnnecessaryDoubleQuotes = (function() {
     if (!stringValue) {
       return false;
     }
+    if (((ref = tokenApi.peek(2)) != null ? ref[0] : void 0) === 'REGEX_END') {
+      return false;
+    }
     hasLegalConstructs = this.isInterpolation || this.hasSingleQuote(tokenValue);
     return !hasLegalConstructs;
   };
@@ -2432,7 +2578,7 @@ module.exports = NoUnnecessaryDoubleQuotes = (function() {
 
 
 
-},{}],36:[function(require,module,exports){
+},{}],38:[function(require,module,exports){
 var NoUnnecessaryFatArrows, any,
   bind = function(fn, me){ return function(){ return fn.apply(me, arguments); }; };
 
@@ -2514,7 +2660,7 @@ module.exports = NoUnnecessaryFatArrows = (function() {
 
 
 
-},{}],37:[function(require,module,exports){
+},{}],39:[function(require,module,exports){
 var NonEmptyConstructorNeedsParens, ParentClass,
   extend = function(child, parent) { for (var key in parent) { if (hasProp.call(parent, key)) child[key] = parent[key]; } function ctor() { this.constructor = child; } ctor.prototype = parent.prototype; child.prototype = new ctor(); child.__super__ = parent.prototype; return child; },
   hasProp = {}.hasOwnProperty;
@@ -2547,7 +2693,7 @@ module.exports = NonEmptyConstructorNeedsParens = (function(superClass) {
 
 
 
-},{"./empty_constructor_needs_parens.coffee":15}],38:[function(require,module,exports){
+},{"./empty_constructor_needs_parens.coffee":15}],40:[function(require,module,exports){
 var RuleProcessor;
 
 module.exports = RuleProcessor = (function() {
@@ -2609,7 +2755,7 @@ module.exports = RuleProcessor = (function() {
 
 
 
-},{}],39:[function(require,module,exports){
+},{}],41:[function(require,module,exports){
 var SpaceOperators,
   indexOf = [].indexOf || function(item) { for (var i = 0, l = this.length; i < l; i++) { if (i in this && this[i] === item) return i; } return -1; };
 
@@ -2714,12 +2860,10 @@ module.exports = SpaceOperators = (function() {
 
 
 
-},{}],40:[function(require,module,exports){
+},{}],42:[function(require,module,exports){
 var RuleProcessor;
 
 module.exports = RuleProcessor = (function() {
-  function RuleProcessor() {}
-
   RuleProcessor.prototype.rule = {
     name: 'spacing_after_comma',
     description: 'This rule requires a space after commas.',
@@ -2727,23 +2871,46 @@ module.exports = RuleProcessor = (function() {
     message: 'Spaces are required after commas'
   };
 
-  RuleProcessor.prototype.tokens = [','];
+  RuleProcessor.prototype.tokens = [',', 'REGEX_START', 'REGEX_END'];
+
+  function RuleProcessor() {
+    this.inRegex = false;
+  }
 
   RuleProcessor.prototype.lintToken = function(token, tokenApi) {
-    if (!(token.spaced || token.newLine)) {
+    var type;
+    type = token[0];
+    if (type === 'REGEX_START') {
+      this.inRegex = true;
+      return;
+    }
+    if (type === 'REGEX_END') {
+      this.inRegex = false;
+      return;
+    }
+    if (!(token.spaced || token.newLine || token.generated || this.isRegexFlag(token, tokenApi))) {
       return {
         context: token[1]
       };
     }
   };
 
+  RuleProcessor.prototype.isRegexFlag = function(token, tokenApi) {
+    var maybeEnd;
+    if (!this.inRegex) {
+      return false;
+    }
+    maybeEnd = tokenApi.peek(3);
+    return (maybeEnd != null ? maybeEnd[0] : void 0) === 'REGEX_END';
+  };
+
   return RuleProcessor;
 
 })();
 
 
 
-},{}],41:[function(require,module,exports){
+},{}],43:[function(require,module,exports){
 var CamelCaseClasses;
 
 module.exports = CamelCaseClasses = (function() {

data/lib/coffeelint.rb

@@ -107,11 +107,11 @@ module Coffeelint
 
   def self.run_test_suite(directory, config = {})
     pretty_output = config.has_key?(:pretty_output) ? config.delete(:pretty_output) : true
-    success = true
+    errors_count = 0
     Coffeelint.lint_dir(directory, config) do |name, errors|
+      errors_count += errors.count
       result = Coffeelint.display_test_results(name, errors, pretty_output)
-      success = false if not result
     end
-    success
+    errors_count
   end
 end

data/lib/coffeelint/version.rb

@@ -1,3 +1,3 @@
 module Coffeelint
-  VERSION = "1.9.1"
+  VERSION = "1.10.0-patch.1"
 end

data/lib/tasks/coffeelint.rake

@@ -1,5 +1,5 @@
 desc "lint application javascript"
 task :coffeelint do
-  success = Coffeelint.run_test_suite('app') and Coffeelint.run_test_suite('spec')
-  fail "Lint!" unless success
+  failures = Coffeelint.run_test_suite('app') + Coffeelint.run_test_suite('spec')
+  fail "Lint!" unless failures == 0
 end

data/spec/config_spec.rb

@@ -51,7 +51,7 @@ module CoffeeLint
           expect(Config.send(:config_files_in_path, [''])).to eq(result)
         end
 
-        it 'builds usefull path segements' do
+        it 'builds useful path segements' do
           result = %w(config/coffeelint.json config/.coffeelint.json)
           expect(Config.send(:config_files_in_path, 'config')).to eq(result)
           expect(Config.send(:config_files_in_path, ['config'])).to eq(result)

metadata

@@ -1,94 +1,83 @@
 --- !ruby/object:Gem::Specification
 name: coffeelint
 version: !ruby/object:Gem::Version
-  version: 1.9.1
-  prerelease: 
+  version: 1.10.0.pre.patch.1
 platform: ruby
 authors:
 - Zachary Bush
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-02-23 00:00:00.000000000 Z
+date: 2015-08-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: coffee-script
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: json
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: execjs
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 3.1.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 3.1.0
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: Ruby bindings for coffeelint
@@ -99,16 +88,17 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .gitmodules
-- .rspec
-- .travis.yml
+- ".gitignore"
+- ".gitmodules"
+- ".rspec"
+- ".travis.yml"
 - Gemfile
 - LICENSE.txt
 - README.md
 - Rakefile
 - bin/coffeelint.rb
 - coffeelint.gemspec
+- coffeelint/lib/coffeelint.js
 - lib/coffeelint.rb
 - lib/coffeelint/cmd.rb
 - lib/coffeelint/config.rb
@@ -119,31 +109,29 @@ files:
 - spec/coffeelint_spec.rb
 - spec/config_spec.rb
 - spec/spec_helper.rb
-- coffeelint/lib/coffeelint.js
 homepage: https://github.com/zipcodeman/coffeelint-ruby
 licenses:
 - MIT
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.23
+rubygems_version: 2.4.5
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: Ruby bindings for coffeelint along with railtie to add rake task to rails
 test_files:
 - spec/assets/.coffeelint.json