Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Commits on May 2, 2012
  1. @rwaldron @valueof

    Don't show unnecessary warnings for new my.Array()

    rwaldron authored valueof committed
    There is no need in showing "Use the _ literal notation." warning
    when Array is simply a property of some custom object. Same with
    Object.
    
    Fixes GH-527.
  2. @valueof
Commits on Apr 11, 2012
  1. @istvan-antal @valueof

    Rhino: convert indent into a number.

    istvan-antal authored valueof committed
    This patch fixes a bug when values of option indent were being
    passed into JSHINT as strings instead of numbers.
    
    Fixes GH-495.
Commits on Apr 7, 2012
  1. @valueof

    Patch from J-Griffin

    valueof authored
    Added recursive directory support to WSH wrapper.
    
    The WSH env now has support for passing in a directory name.
    
    The only functional change that was made for single files is that the
    passed script name is spit out in the output just like when running
    against a directory. Let me know if that is a problem and I can make a
    change.
    
    + Fixes GH-471
    
    * (merged manually)
  2. @valueof

    Added (hidden) r2config

    valueof authored
  3. @valueof

    Merge GH-508

    valueof authored
    Although JSHint currently recognizes the `/*jslint` option comment syntax,
    several of the options have either changed names or inverted meanings
    between JSLint and JSHint.  This creates many needless errors when running
    JSHint against a project using the JSLint option comments.
    To increase compatibility and ease transitions from JSLint to JSHint,
    these patches translate JSLint options into JSHint options where possible
    and unambiguous.
    
    + Fixes GH-508
    
    * patch-508:
      Add test for compatibility with renamed JSLint options
      Support compatibility with renamed JSLint options
      Add test for compatibility with inverted JSLint options
      Support compatibility with inverted JSLint options
  4. @kevinoid @valueof

    Add test for compatibility with renamed JSLint options

    kevinoid authored valueof committed
    Create jslintRenamed.js unit test which tests handling of the
    "/*jslint" eqeq option.  If the option name is not converted from "eqeq"
    to "eqeqeq" and the value inverted, the test will fail.
    
    Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
  5. @kevinoid @valueof

    Support compatibility with renamed JSLint options

    kevinoid authored valueof committed
    Several of the options inherited from JSLint have retained their
    original function but changed their names.  Since the "/*jslint" option
    syntax is still supported, running JSHint on a project using the
    "/*jslint" option syntax with these options has no effect and may either
    result in errors the developer has already reviewed or miss errors that
    have been specifically requested.
    
    To support this use case and increase compatibility with JSLint, catch
    such renamed options and store the values with the corresponding JSHint
    option when encountered with the "/*jslint" option syntax.
    
    Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
  6. @kevinoid @valueof

    Add test for compatibility with inverted JSLint options

    kevinoid authored valueof committed
    Create jslintInverted.js unit test which tests handling of the
    "/*jslint" plusplus option.  If this is parsed the same as a "/*jshint"
    option the test will fail.
    
    Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
  7. @kevinoid @valueof

    Support compatibility with inverted JSLint options

    kevinoid authored valueof committed
    Several of the options inherited from JSLint have retained their
    original names but inverted their meanings.  Since the "/*jslint" option
    syntax is still supported, running JSHint on a project using the
    "/*jslint" option syntax with these options produces a result that is
    opposite to the intentions of the developer.
    
    To support this use case and increase compatibility with JSLint, invert
    such option values when encountered with the "/*jslint" option syntax.
    
    Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
Commits on Apr 6, 2012
  1. @valueof

    Merge GH-509

    valueof authored
    As a convenience to users, it would be nice if `jshint-rhino.js` were directly runnable rather than requiring the user to invoke rhino and pass the script as an argument for every use of JSHint.  To that end, these commits add a shebang line and mark `jshint-rhino.js` executable during build.
    
    + Fixes GH-509
    
    * patch-509:
      Make jshint-rhino.js runnable
  2. @kevinoid @valueof

    Make jshint-rhino.js runnable

    kevinoid authored valueof committed
    Add a shebang to jshint-rhino.js to tell the shell how to execute the
    file and mark it executable after build.
    
    Note1:  Ignore the return code of chmod in case the user is on a
    filesystem which does not support setting execute permission.
    Note2:  Make use  of /usr/bin/env to catch rhino anywhere in the PATH.
    
    Also reformat the rhino build target slightly to match the other targets
    (replace chained '&&' with separate shell commands).  Combine multiple
    invocations of cat into one.
    
    Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
  3. @valueof
Commits on Mar 28, 2012
  1. @goatslacker

    Merge pull request #497 from brentlintner/copy.option.object

    goatslacker authored
    Internally mutating the option argument causes unexpected behaviour.
Commits on Mar 27, 2012
  1. @brentlintner

    Internally mutating the option argument causes unexpected behaviour.

    brentlintner authored
    Example:
    
        var options = {regexp: true};
        JSHINT("/*jshint regexp: false */\nvar c = /a.b/;", options);
        JSHINT("var a = /d.e/;", options);
    
    In the second call, `options.regexp == false`, and no errors are logged.
Commits on Mar 13, 2012
  1. @goatslacker

    Merge pull request #472 from jshint/gh395

    goatslacker authored
    Warn about invalid (often typoed) options.
Commits on Mar 12, 2012
  1. @valueof
  2. @goatslacker

    Merge pull request #476 from wagenet/object-function

    goatslacker authored
    Allows Object() to be called as a function
  3. @goatslacker

    Merge pull request #479 from fabiomcosta/patch-1

    goatslacker authored
    Adds column to Trailing Whitespace warning.
Commits on Mar 11, 2012
  1. @fabiomcosta

    changing the fix for showing the column of the trailing whitespace, b…

    fabiomcosta authored
    …ut does the same thing. Changing tests to check for the column number.
Commits on Mar 10, 2012
  1. @wagenet
  2. @fabiomcosta
Commits on Mar 7, 2012
  1. @valueof
  2. @WolfgangKluge @valueof

    Support for the 'with' statement.

    WolfgangKluge authored valueof committed
    This patch adds an opt-in support for the 'with' statement. By
    default JSHint warns against using 'with' but this warning
    can be overriden with option 'withstmt'.
    
    In strict mode, 'with' is _always_ prohibited and results in an
    error message from JSHint.
Commits on Mar 6, 2012
  1. @goatslacker

    Merge pull request #473 from jshint/gh377

    goatslacker authored
    Assume globalstrict in the node environment.
Commits on Mar 5, 2012
  1. @valueof

    Assume globalstrict in the node environment.

    valueof authored
    Fixes GH-377. Since there're no problems with using module level
    "use strict" in Node programs we don't need to display a warning
    about it.
    
    Test plan: `make test`.
  2. @valueof

    Warn about invalid (often typoed) options.

    valueof authored
    Fixes GH-395.
    
    I've added code that checks both options provided as arguments to the
    JSHINT function and as command comments (`/*jshint ... */`). If JSHint
    doesn't know about an option it produces a warning:
    
      Bad option: '<name>'.
    
    Test plan: `make test`.
  3. @valueof
Commits on Mar 4, 2012
  1. @J-Griffin
Commits on Mar 3, 2012
  1. @goatslacker

    Merge pull request #469 from WolfgangKluge/reset-values

    goatslacker authored
    Reset value `comma.first` on every run.
  2. @WolfgangKluge
  3. @WolfgangKluge
Commits on Mar 2, 2012
  1. @goatslacker
Commits on Feb 27, 2012
  1. @goatslacker
  2. @goatslacker
Something went wrong with that request. Please try again.