Skip to main content

Bug Tracker

Side navigation

#13755 closed bug (fixed)

Opened April 09, 2013 01:10PM UTC

Closed April 09, 2013 04:02PM UTC

Last modified April 09, 2013 04:52PM UTC

Update .jshintrc to match style guide

Reported by: scottgonzalez Owned by: timmywil
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.

Attachments (0)
Change History (4)

Changed April 09, 2013 01:44PM UTC by timmywil comment:1

component: unfiledbuild
milestone: None1.10/2.0
owner: → timmywil
priority: undecidedlow
status: newassigned

Changed April 09, 2013 04:02PM UTC by Timmy Willison comment:2

resolution: → fixed
status: assignedclosed

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

Changeset: 0fa52c11cbfe70780648b99717f1dd3502befaff

Changed April 09, 2013 04:22PM UTC by timmywil comment:3

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.

Changed April 09, 2013 04:52PM UTC by timmywil comment:4

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

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