Skip to:
Content

bbPress.org

Changes between Initial Version and Version 3 of Ticket #2746


Ignore:
Timestamp:
02/28/2015 04:44:56 AM (11 years ago)
Author:
netweb
Comment:

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #2746

    • Property Status changed from new to closed
    • Property Component changed from General to Build/Test Tools
    • Property Summary changed from Remove jshint 'trailing' option from .jshintrc to Remove deprecated jshint 'onevar" and 'trailing' option from .jshintrc
    • Property Milestone changed from Awaiting Review to 2.6
    • Property Owner set to netweb
    • Property Resolution changed from to fixed

  • Ticket #2746 – Description

    initial v3  
    1 As of JSHint v2.5 option `trailing` is no longer an option.
     1Via #WP28236
     2
     3A few things here....
     4
     5Since we are now using JSHint v2.5 via `grunt-contrib-jshint` v0.10.0 in our `package.json` the `onevar` and `trailing` options in `.jshintrc` are pointless, support for both of these was removed in JSHint v2.5.
     6
     7I've had my head in JSCS for the past couple of days and I'll create a JSCS ticket and link back here once done to investigate JSCS options further and to add support for these now removed `onevar` and `trailing` options via JSCS.
     8
     9JSCS includes equivalent options for ease of migrating settings from JSHint to JSCS
     10
     11* `onevar` - http://jscs.info/rules.html#requiremultiplevardecl
     12* `trailing` - http://jscs.info/rules.html#disallowtrailingwhitespace
    213
    314See https://github.com/jshint/jshint/pull/1554