jish/pre-commit

View on GitHub

Showing 171 of 259 total issues

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 (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 (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

                                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 (!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 (xmode === 'html') {
                                            if (option.safe) {
                                                warningAt("ADsafe string violation.",
                                                        line, character + j);
                                            }
            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 (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

                                          if (temp.length == 1){
                                              reader.reset();
                                          } else {
                                              value += temp;
                                          }
                  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[0].value === 0)){
                                                    reporter.report("Using height 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 (!(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 (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 (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

                                                            for (;;) {
                                                                c = s.charAt(0);
                                                                character += 1;
                                                                s = s.substr(1);
                                                                if (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 (left.value === 'Math') {
                                                          warning("Math is not a function.", left);
                                                      } else if (option.newcap) {
                                                          warning(
                              "Missing 'new' prefix when invoking a constructor.", left);
                              Severity: Major
                              Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

                                Avoid deeply nested control flow statements.
                                Open

                                                            if (CSSLint.Util.indexOf(actual, item) === -1) {
                                                                propertiesSpecified = (actual.length === 1) ? actual[0] : (actual.length == 2) ? actual.join(" and ") : actual.join(", ");
                                                                reporter.report("The property " + item + " is compatible with " + propertiesSpecified + " and should be included as well.", value.actualNodes[0].line, value.actualNodes[0].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 === '(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 (nexttoken.id !== q) {
                                                                error("Missing close quote on script attribute.");
                                                            }
                                    Severity: Major
                                    Found in lib/pre-commit/support/jslint/lint.js - About 45 mins to fix

                                      Avoid deeply nested control flow statements.
                                      Open

                                                                  if (tokenStream.LA(3) == Tokens.EQUALS && this.options.ieFilters){
                                                                      value = this._ie_function();
                                                                  } else {
                                                                      value = this._function();
                                                                  }
                                      Severity: Major
                                      Found in lib/pre-commit/support/csslint/csslint.js - About 45 mins to fix

                                        Avoid deeply nested control flow statements.
                                        Open

                                                            if (ValidationTypes.isAny(expression, xDir + " | " + yDir)) {
                                                                result = true;
                                                                ValidationTypes.isAny(expression, numeric);
                                                            }
                                        Severity: Major
                                        Found in lib/pre-commit/support/csslint/csslint.js - About 45 mins to fix
                                          Severity
                                          Category
                                          Status
                                          Source
                                          Language