Upgrade typescript linter

Summary:
Just shows some warnings when running `yarn lint` otherwise about lack of compatibility.

Required updating the rules because of a breaking change here: https://github.com/typescript-eslint/typescript-eslint/pull/3800/files

Reviewed By: lblasa

Differential Revision: D33533052

fbshipit-source-id: fd73d534bbd6e1933321b5811832b05a9739af95

desktop/eslint-plugin-flipper/package.json

@@ -10,8 +10,8 @@
   "license": "MIT",
   "bugs": "https://github.com/facebook/flipper/issues",
   "dependencies": {
-    "@typescript-eslint/experimental-utils": "^4.33.0",
-    "@typescript-eslint/parser": "^4.33.0",
+    "@typescript-eslint/experimental-utils": "^5.9.1",
+    "@typescript-eslint/parser": "^5.9.1",
     "fs-extra": "^10.0.0"
   },
   "devDependencies": {},

desktop/eslint-plugin-flipper/src/rules/noConsoleErrorWithoutContext.ts

@@ -21,7 +21,6 @@ export default createESLintRule<Options, MessageIds>({
     type: 'problem',
     docs: {
       description: 'TBD',
-      category: 'Possible Errors',
       recommended: 'warn',
     },
     schema: [],

desktop/eslint-plugin-flipper/src/rules/noElectronRemoteImports.ts

@@ -21,7 +21,6 @@ export default createESLintRule<Options, MessageIds>({
     type: 'suggestion',
     docs: {
       description: '`remote` is slow. Please be careful when using it.',
-      category: 'Possible Errors',
       recommended: 'warn',
     },
     schema: [],

desktop/eslint-plugin-flipper/src/rules/noRelativeImportsAcrossPackages.ts

@@ -38,7 +38,6 @@ export default createESLintRule<Options, MessageIds>({
     type: 'problem',
     docs: {
       description: `Ensure that package boundaries are respected within monorepo`,
-      category: 'Possible Errors',
       recommended: 'error',
     },
     schema: [],