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: consolidate ESLint dependencies #40995

Closed
wants to merge 4 commits into from
Closed

tools: consolidate ESLint dependencies #40995

wants to merge 4 commits into from

Commits on Nov 29, 2021

  1. tools: run ESLint update to minimize diff on subsequent update 

    PR-URL: nodejs#40995
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
    @Trott
    Trott committed Nov 29, 2021
    Configuration menu
    Copy the full SHA
    45422db View commit details
    Browse the repository at this point in the history

  2. tools: update ESLint update script to consolidate dependencies 

    PR-URL: nodejs#40995
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
    @Trott
    Trott committed Nov 29, 2021
    Configuration menu
    Copy the full SHA
    6fd0a60 View commit details
    Browse the repository at this point in the history

  3. tools: consolidate ESLint dependencies 

    PR-URL: nodejs#40995
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
    @Trott
    Trott committed Nov 29, 2021
    Configuration menu
    Copy the full SHA
    4c41fea View commit details
    Browse the repository at this point in the history

  4. tools: remove unneeded tool in update-eslint.sh 

    The tool to remove absolute paths from package.json files is no longer
necessary. It appears that npm no longer stores these paths, or at least
not in a way that causes the kind of churn we saw in the past.

PR-URL: nodejs#40995
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
    @Trott
    Trott committed Nov 29, 2021
    Configuration menu
    Copy the full SHA
    6d8ab76 View commit details
    Browse the repository at this point in the history