[wp-trac] [WordPress Trac] #33021: Global javascript variables incorrectly defined

[WordPress Trac]

#33021: Global javascript variables incorrectly defined
------------------------------+------------------------------
 Reporter:  johnjamesjacoby   |       Owner:
     Type:  defect (bug)      |      Status:  new
 Priority:  normal            |   Milestone:  Awaiting Review
Component:  Build/Test Tools  |     Version:
 Severity:  normal            |  Resolution:
 Keywords:  has-patch         |     Focuses:  javascript
------------------------------+------------------------------

Comment (by netweb):

 I'm going to endorse a third option to this discussion, `eslint` I plan on
 (and will create a new ticket) for us to switch to ESLint which will
 replace `JSHint` and `JSValidate` (I'll outline the benefits for us in
 that new ticket).

 The ESLint docs here: http://eslint.org/docs/user-guide/configuring
 #specifying-globals

 These docs also match up with what John has stated above, `globals` in the
 config files, and `global` in the .js files.

 I'm +1 on this proposed change, it fixes the IDE issues, inline with our
 own JS Docs, and aligns with best practices.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/33021#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform