Sign upLogin

jish/pre-commit

View on GitHub
Star

Showing 171 of 259 total issues

Avoid deeply nested control flow statements.
Open

                    if (typeof b !== 'number' || !isFinite(b) || b <= 0 ||
                            Math.floor(b) !== b) {
                        error("Expected a small integer and instead saw '{a}'.",
                                v, v.value);
                    }
Severity: Major
Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

    Avoid deeply nested control flow statements.
    Open

                            if (option.undef) {
                            warning("'{a}' is not defined.", token, v);
                        } else {
                            funct[v] = true;
                        }
    Severity: Major
    Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

      Function warning has 6 arguments (exceeds 4 allowed). Consider refactoring.
      Open

          function warning(m, t, a, b, c, d) {
      Severity: Minor
      Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

        Avoid deeply nested control flow statements.
        Open

                                        if (!nextLine()) {
                                    errorAt("Unclosed HTML comment.", l, c);
                                }
        Severity: Major
        Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

          Avoid deeply nested control flow statements.
          Open

                          } else if (t.value === 'maxlen' && o === '/*jslint') {
                    b = +v.value;
                    if (typeof b !== 'number' || !isFinite(b) || b <= 0 ||
                            Math.floor(b) !== b) {
                        error("Expected a small integer and instead saw '{a}'.",
          Severity: Major
          Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

            Avoid deeply nested control flow statements.
            Open

                                } else if (a === 'style') {
                        xmode = 'scriptstring';
                        advance('=');
                        q = nexttoken.id;
                        if (q !== '"' && q !== "'") {
            Severity: Major
            Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

              Avoid deeply nested control flow statements.
              Open

                                          if (modifier.type == "attribute"){
                                if (/([\~\|\^\$\*]=)/.test(modifier)){
                                    reporter.report("Attribute selectors with " + RegExp.$1 + " are slow!", modifier.line, modifier.col, rule);
                                }
                            }
              Severity: Major
              Found in lib/pre-commit/support/csslint/csslint.js - About 45 mins to fix

                Avoid deeply nested control flow statements.
                Open

                        } else if (a === 'name') {
            if (option.adsafe && v.indexOf('_') >= 0) {
                warning("ADsafe name violation.");
            }
        }
                Severity: Major
                Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

                  Avoid deeply nested control flow statements.
                  Open

                                                  if (combinator !== null){
                                    selector.push(combinator);
                                } else {
                                    selector.push(ws);
                                }
                  Severity: Major
                  Found in lib/pre-commit/support/csslint/csslint.js - About 45 mins to fix

                    Avoid deeply nested control flow statements.
                    Open

                                                    for (;;) {
                                    c = s.charAt(0);
                                    if ((c < '0' || c > '9') &&
                                            (c < 'a' || c > 'f') &&
                                            (c < 'A' || c > 'F')) {
                    Severity: Major
                    Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

                      Avoid deeply nested control flow statements.
                      Open

                                              if (q !== '"' && q !== "'") {
                            error("Missing quote.");
                        }
                      Severity: Major
                      Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

                        Avoid deeply nested control flow statements.
                        Open

                                                    if (/([^\)]+)\(/.test(parts[i])){
                                colorType = RegExp.$1.toUpperCase();
                            }
                        Severity: Major
                        Found in lib/pre-commit/support/csslint/csslint.js - About 45 mins to fix

                          Avoid deeply nested control flow statements.
                          Open

                                                  if (!option.on) {
                            warning("Avoid HTML event handlers.");
                        }
                          Severity: Major
                          Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

                            Avoid deeply nested control flow statements.
                            Open

                                                        if (classCount > 1){
                                reporter.report("Don't use adjoining classes.", part.line, part.col, rule);
                            }
                            Severity: Major
                            Found in lib/pre-commit/support/csslint/csslint.js - About 45 mins to fix

                              Avoid deeply nested control flow statements.
                              Open

                                                          if (part.elementName && modifier.type == "id"){
                                reporter.report("Element (" + part + ") is overqualified, just use " + modifier + " without element name.", part.line, part.col, rule);
                            } else if (modifier.type == "class"){

                                if (!classes[modifier]){
                              Severity: Major
                              Found in lib/pre-commit/support/csslint/csslint.js - About 45 mins to fix

                                Avoid deeply nested control flow statements.
                                Open

                                                                if (combinator !== null){
                                    this._unexpectedToken(tokenStream.LT(1));
                                }
                                Severity: Major
                                Found in lib/pre-commit/support/csslint/csslint.js - About 45 mins to fix

                                  Avoid deeply nested control flow statements.
                                  Open

                                                              if (!lastProperty || (lastProperty.property.text.toLowerCase() != name || lastProperty.colorType != "compat")){
                                reporter.report("Fallback " + name + " (hex or RGB) should precede " + colorType + " " + name + ".", event.line, event.col, rule);
                            }
                                  Severity: Major
                                  Found in lib/pre-commit/support/csslint/csslint.js - About 45 mins to fix

                                    Avoid deeply nested control flow statements.
                                    Open

                                                            if (nexttoken.id === ')') {
                            warning("Use the array literal notation [].", token);
                        } else {
                            i = parse(0);
                            c.dimension = i;
                                    Severity: Major
                                    Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

                                      Function error has 6 arguments (exceeds 4 allowed). Consider refactoring.
                                      Open

                                          function error(m, t, a, b, c, d) {
                                      Severity: Minor
                                      Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

                                        Avoid deeply nested control flow statements.
                                        Open

                                                                    if (!propertyGroups[prop]) {
                                propertyGroups[prop] = {
                                    full : variations.slice(0),
                                    actual : [],
                                    actualNodes: []
                                        Severity: Major
                                        Found in lib/pre-commit/support/csslint/csslint.js - About 45 mins to fix
                                          Severity
                                          Category
                                          Status
                                          Source
                                          Language