Skip to:

Opened 5 years ago

Closed 5 years ago

Last modified 4 years ago

#6265 closed defect (bug) (fixed)

Remove deprecated JSHint `onevar` and `trailing` arguments

Reported by: netweb Owned by: djpaul
Milestone: 2.3 Priority: normal
Severity: normal Version:
Component: Build/Test Tools Keywords: has-patch


Via #WP28236 (More context and discussion if curious)

Since 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.

I'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.

JSCS includes equivalent options for ease of migrating settings from JSHint to JSCS

Attachments (1)

6265.diff (309 bytes) - added by netweb 5 years ago.

Download all attachments as: .zip

Change History (3)

5 years ago

#1 @djpaul
5 years ago

  • Owner set to djpaul
  • Resolution set to fixed
  • Status changed from new to closed

In 9565:

Tools: remove deprecated and unsupported parameters from the JSHint configuration.

Re-instating support for these options will likely come with a switch to a different JS code linter.
See #6265 for details.

Fixes #6265, props netweb

#2 @DJPaul
4 years ago

  • Component changed from Tools - Build Process to Build/Test Tools
Note: See TracTickets for help on using tickets.