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 8.0.0 #40394

Merged
merged 7 commits into from Oct 12, 2021
Merged

tools: update ESLint to 8.0.0 #40394

merged 7 commits into from Oct 12, 2021

Commits on Oct 12, 2021

  1. tools: fix ESLint update scripts 

    Changes to the npm dependency in the core repo require changes to
update-eslint.sh and update-bable-eslint.sh.

PR-URL: nodejs#40394
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
    @Trott
    Trott committed Oct 12, 2021
    Copy the full SHA
    0a1b399 View commit details
    Browse the repository at this point in the history

  2. tools: prepare ESLint rules for 8.0.0 requirements 

    PR-URL: nodejs#40394
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
    @Trott
    Trott committed Oct 12, 2021
    Copy the full SHA
    dd8e219 View commit details
    Browse the repository at this point in the history

  3. tools: update ESLint to 8.0.0 

    PR-URL: nodejs#40394
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
    @Trott
    Trott committed Oct 12, 2021
    Copy the full SHA
    8ca76eb View commit details
    Browse the repository at this point in the history

  4. tools: remove @babel/plugin-syntax-top-level-await 

    ESLint 8.0.0 introduced support for top level await, so
@babel/plugin-syntax-top-level-await is no longer necessary.

PR-URL: nodejs#40394
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
    @Trott
    Trott committed Oct 12, 2021
    Copy the full SHA
    a6fd39f View commit details
    Browse the repository at this point in the history

  5. tools: remove @bable/plugin-syntax-class-properties 

    ESLint 8.0.0 makes the plugin unnecessary.

PR-URL: nodejs#40394
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
    @Trott
    Trott committed Oct 12, 2021
    Copy the full SHA
    b39db95 View commit details
    Browse the repository at this point in the history

  6. tools: remove @babel/plugin-syntax-import-assertions 

    As of ESLint 8.0.0 at least, this plugin is unnecessary.

PR-URL: nodejs#40394
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
    @Trott
    Trott committed Oct 12, 2021
    Copy the full SHA
    43c780e View commit details
    Browse the repository at this point in the history

  7. tools: udpate @babel/eslint-parser 

    Remove unused plugins and update @babel/eslint-parser

PR-URL: nodejs#40394
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
    @Trott
    Trott committed Oct 12, 2021
    Copy the full SHA
    012152d View commit details
    Browse the repository at this point in the history