Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

tools: update ESLint to 3.3.0 and enable rules #8097

Closed
wants to merge 2 commits into from
Closed

tools: update ESLint to 3.3.0 and enable rules #8097

wants to merge 2 commits into from

Commits on Aug 13, 2016

  1. test: remove extraneous space 

    Change `require ('zlib');` to `require('zlib');` in conformance with
style in the rest of the code base. This is in preparation for enabling
linting for that issue.
    @Trott
    Trott committed Aug 13, 2016
    Configuration menu
    Copy the full SHA
    ae3b848 View commit details
    Browse the repository at this point in the history

  2. tools: update ESLint to 3.3.0 and enable rules 

    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
    @Trott
    Trott committed Aug 13, 2016
    Configuration menu
    Copy the full SHA
    1e2e735 View commit details
    Browse the repository at this point in the history