Adding ability to pass rules-max limit using cli

Author: hrobertking

src/cli/common.js

@@ -20,7 +20,8 @@ function cli(api) {
         "exclude-list": { "format" : "<file|dir[,file|dir]+>", "description" : "Indicate which files/directories to exclude from being linted." },
         "config"      : { "format" : "<file>",                 "description" : "Reads csslint options from specified file." },
         "version"     : { "format" : "",                       "description" : "Outputs the current version number." },
-        "name-pattern": { "format" : "<pattern>",              "description" : "Indicate which pattern naming should follow." }
+        "name-pattern": { "format" : "<pattern>",              "description" : "Indicate which pattern naming should follow." },
+        "max-rules"   : { "format" : "<count>",                "description" : "The maximum number of rules allowed in a stylesheet." }
     };
 
     //-------------------------------------------------------------------------

src/rules/rules-max.js

@@ -6,21 +6,25 @@ CSSLint.addRule({
 
     // rule information
     id: "rules-max",
-    name: "Warn when 500+ rules are present",
-    desc: "Will warn when the rule count in a stylesheet crosses the 500 threshold.",
-    browsers: "IE",
+    name: "Warn when too many rules are present",
+    desc: "Will warn when the rule count in a stylesheet crosses the limit specified.",
+    browsers: "All",
 
     // initialization
-    init: function(parser, reporter) {
+    init: function(parser, reporter, options) {
         "use strict";
-        var rule = this, ruleCount = 0, MAX_RULES = 499;
+        var rule = this,
+            ruleCount = 0,
+            limit = options ? options["max-rules"] : 0;
+
+        limit = limit || 0;
 
         parser.addListener("startrule", function() {
             ruleCount += 1;
         });
 
         parser.addListener("endstylesheet", function() {
-            if (ruleCount > MAX_RULES) {
+            if (limit && limit < ruleCount) {
                 reporter.report("There are too many rules (" + ruleCount + ") in the stylesheet.", 0, 0, rule);
             }
         });

tests/rules/rules-max.js

@@ -39,8 +39,8 @@
             Assert.areEqual(0, result.messages.length);
         },
 
-        "Using 500 or more rules should result in a warning": function() {
-            var result = CSSLint.verify(this.cssB, { "rules-max": 1 });
+        "Using more rules than the limit should result in a warning": function() {
+            var result = CSSLint.verify(this.cssB, { "rules-max": 1 }, { "max-rules": MAX_RULES });
 
             Assert.areEqual(1, result.messages.length);
             Assert.areEqual("warning", result.messages[0].type);