[core] Update @typescript-eslint/* packages and remove deprecated eslint-config-airbnb-typescript package

[ZeeshanTamboli]

Resolves #44532

Part of mui/mui-public#264

I have applied the rules from eslint-config-airbnb-typescript here.

For the record, these style rules which were applied in eslint-config-airbnb-typescript are no longer supported in @typescript-eslint/eslint-plugin and they have moved it to stylistic:

• comma-dangle
• brace-style
• comma-spacing
• func-call-spacing
• indent
• keyword-spacing
• lines-between-class-members
• no-extra-parens
• no-extra-semi
• space-before-blocks
• quotes
• semi
• space-before-function-paren
• space-infix-ops
• object-curly-spacing

These should be handle by Prettier.

Questions:

1. Should I not use eslint-config-airbnb-base and directly hardcode the rules here? The rules options will be the same.
2. Should I use @typescript-eslint' s recommended preset?
3. Will this affect other repos like MUI X?

[mui-bot]

Netlify deploy preview

https://deploy-preview-45245--material-ui.netlify.app/

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against ed03594

[ZeeshanTamboli]

Override caughtErrors to none here to avoid an unused error parameter eslint error in the catch block. It's a breaking change in v8—the default is changed to all. See https://typescript-eslint.io/blog/announcing-typescript-eslint-v8/#rule-breaking-changes. So I had to override it.

[Janpot]

Just to get us started on properly abstracting the basic rules, can this go in its own file and use the extends feature?

diff --git a/.eslintrc.js b/.eslintrc.js
index 24f8b126a7..7b402f1b33 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -64,7 +64,7 @@ module.exports = /** @type {Config} */ ({
     'plugin:eslint-plugin-import/recommended',
     'plugin:eslint-plugin-import/typescript',
     'eslint-config-airbnb',
-    'eslint-config-airbnb-typescript',
+    './eslint/config-airbnb-typescript.js',
     'eslint-config-prettier',
   ],
   parser: '@typescript-eslint/parser',

[ZeeshanTamboli]

Just to get us started on properly abstracting the basic rules, can this go in its own file and use the extends feature?

diff --git a/.eslintrc.js b/.eslintrc.js
index 24f8b126a7..7b402f1b33 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -64,7 +64,7 @@ module.exports = /** @type {Config} */ ({
     'plugin:eslint-plugin-import/recommended',
     'plugin:eslint-plugin-import/typescript',
     'eslint-config-airbnb',
-    'eslint-config-airbnb-typescript',
+    './eslint/config-airbnb-typescript.js',
     'eslint-config-prettier',
   ],
   parser: '@typescript-eslint/parser',

@Janpot Done.

[Janpot]

👍 Thanks, let's go with this for now. We'll revisit the abstractions once we move to flat config