Bug Tracker

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#13755 closed bug (fixed)

Update .jshintrc to match style guide

Reported by: scottgonzalez Owned by: Timmy Willison
Priority: low Milestone: 1.10/2.0
Component: build Version: 1.9.1
Keywords: Cc:
Blocked by: Blocking:

Description

See 1. Linting in the JS style guide.

Change History (4)

comment:1 Changed 10 years ago by Timmy Willison

Component: unfiledbuild
Milestone: None1.10/2.0
Owner: set to Timmy Willison
Priority: undecidedlow
Status: newassigned

comment:2 Changed 10 years ago by Timmy Willison

Resolution: fixed
Status: assignedclosed

Update jshintrc to conform to new style guide. Conform to onevar and unused in tests. Fixes #13755.

Changeset: 0fa52c11cbfe70780648b99717f1dd3502befaff

comment:3 Changed 10 years ago by Timmy Willison

We can't do onevar in core src because we lint the concatenated file and we shouldn't place variables specific to certain modules somewhere besides the module itself. However, onevar now works in the gruntfile and in the tests.

comment:4 Changed 10 years ago by Timmy Willison

Added onevar to jshintrc, but removed it for the concatenated file:

https://github.com/jquery/jquery/commit/738b8f84310f3d33d22fae976b83d94a684e01a8

Note: See TracTickets for help on using tickets.