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 comment:1
component: | unfiled → build |
---|---|
milestone: | None → 1.10/2.0 |
owner: | → timmywil |
priority: | undecided → low |
status: | new → assigned |
Changed April 09, 2013 04:02PM UTC by comment:2
resolution: | → fixed |
---|---|
status: | assigned → closed |
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 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 comment:4
Added onevar to jshintrc, but removed it for the concatenated file:
https://github.com/jquery/jquery/commit/738b8f84310f3d33d22fae976b83d94a684e01a8