Skip to content

tools: update ESLint to 3.3.0 and enable rules

Rodrigo Muino Tomonari requested to merge github/fork/Trott/eslint3.3.0 into master
Checklist
  • make -j4 test (UNIX), or vcbuild test nosign (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

tools test

Description of change
Update ESLint 3.3.0 and update .eslintrc:

* replace deprecated no-negated-in-lhs rule with no-unsafe-negation
  * http://eslint.org/docs/rules/no-negated-in-lhs
  * http://eslint.org/docs/rules/no-unsafe-negation
* enable no-template-curly-in-string
  * http://eslint.org/docs/rules/no-template-curly-in-string
* enable no-global-assign
  * http://eslint.org/docs/rules/no-global-assign
* enable func-call-spacing
  * http://eslint.org/docs/rules/func-call-spacing

In one test, change require ('zlib'); to require('zlib'); in conformance with style in the rest of the code base. This is to conform with func-call-spacing and the prevailing style in the rest of the code base.

Merge request reports