jish/pre-commit

View on GitHub

Showing 259 of 259 total issues

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

    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 (i <= 0) {
                              error("A '<{a}>' must be within '<{b}>'.",
                                      token, n, 'body');
                          }
      Severity: Major
      Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

        Avoid deeply nested control flow statements.
        Open

                                if (nexttoken.id === '(end)' || nexttoken.id === '(error)') {
                                    error("Missing '>'.", nexttoken);
                                }
        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.newcap && (i < 'A' || i > 'Z')) {
                                      warning(
                              "A constructor name should start with an uppercase letter.",
                                          token);
                                  }
          Severity: Major
          Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

            Avoid deeply nested control flow statements.
            Open

                                            if (d.isDigit()) {
                                                if (token.id !== '.' && xmode !== 'styleproperty') {
                                                    warningAt("Don't use extra leading zeros '{a}'.",
                                                        line, character, t);
                                                }
            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 (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

                  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 (!(prop == "padding" && value.parts.length === 2 && value.parts[1].value === 0)){
                                                    reporter.report("Using width with " + prop + " can sometimes make elements larger than you expect.", properties[prop].line, properties[prop].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 (modifier.type == "class"){
                                                      classCount++;
                                                  }
                      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 (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

                                                            for (;;) {
                                                                b = true;
                                                                c = s.charAt(l);
                                                                l += 1;
                                                                switch (c) {
                            Severity: Major
                            Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

                              Avoid deeply nested control flow statements.
                              Open

                                                              if (i >= 0) {
                                                                  break;
                                                              }
                              Severity: Major
                              Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

                                Avoid deeply nested control flow statements.
                                Open

                                                                if (i >= 0) {
                                                                    errorAt("Nested HTML comment.",
                                                                        line, character + i);
                                                                }
                                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 (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

                                      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 (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
                                          Severity
                                          Category
                                          Status
                                          Source
                                          Language