Opened 12 years ago
Closed 12 years ago
#8557 closed bug (invalid)
jQuery does not pass jslint
| Reported by: | etiger13 | Owned by: | etiger13 |
|---|---|---|---|
| Priority: | undecided | Milestone: | 1.next |
| Component: | build | Version: | 1.5.1 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
There are a few places where jQuery does not pass jslint. Mostly it is === vs == and !== vs !=
Change History (4)
comment:1 Changed 12 years ago by
comment:2 Changed 12 years ago by
| Component: | unfiled → build |
|---|---|
| Owner: | set to etiger13 |
| Status: | new → pending |
Can you link the PR?
comment:3 Changed 12 years ago by
When you do a make on the current master it runs jslint and I get no errors. (Take a look at https://github.com/jquery/jquery/blob/master/build/jslint-check.js to see the filtering we do.) It sounds like you're running bare jslint against the uncompressed jquery.js file?
comment:4 Changed 12 years ago by
| Resolution: | → invalid |
|---|---|
| Status: | pending → closed |
Yeah, we are passing our modified JSLint. More details here: http://docs.jquery.com/JQuery_Core_Style_Guidelines#JSLint
Note: See
TracTickets for help on using
tickets.
Cleaned up and sent in a pull request