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 v1.x #2286

Closed
wants to merge 6 commits into from
Closed

tools: update eslint to v1.x #2286

wants to merge 6 commits into from

Commits on Jan 13, 2016

  1. gitignore: never ignore debug module 

    On case-insensitive platorms, the Debug/ rule catches the debug module
under npm and eslint.

PR-URL: nodejs#2286
    @targos
    targos committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    ab8bf2e View commit details
    Browse the repository at this point in the history

  2. tools: update eslint to v1.10.3

    @targos
    targos committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    431f312 View commit details
    Browse the repository at this point in the history

  3. tools: update eslint config 

    The no-reserved-keys rule doesn't exist anymore and we don't need ES3
compatibility.
escape and unescape are now known by eslint.
--reset flag was removed and it is now the default behavior.

PR-URL: nodejs#2286
    @targos
    targos committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    7a63521 View commit details
    Browse the repository at this point in the history

  4. lib: fix style issues after eslint update 

    With an indentation style of two spaces, it is not possible to indent
multiline variable declarations by four spaces. Instead, the var keyword
is used on every new line.
Use const instead of var where applicable for changed lines.
    @targos
    targos committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    e8bf57a View commit details
    Browse the repository at this point in the history

  5. tools: fix style issue after eslint update

    @targos
    targos committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    a049c91 View commit details
    Browse the repository at this point in the history

  6. test: fix style issues after eslint update 

    Replace var keyword with const or let.
    @targos
    targos committed Jan 13, 2016
    Configuration menu
    Copy the full SHA
    694c280 View commit details
    Browse the repository at this point in the history