From 33dcd4645d8812773a28fe8f38bb4db8e161b9a0 Mon Sep 17 00:00:00 2001
From: Valery Bugakov <skymk1@gmail.com>
Date: Mon, 9 Oct 2023 04:32:26 -0700
Subject: [PATCH] feat: disable `id-length` (#290)

Disabling the `id-length` rule because it was manually disabled in consumer repos (sourcegraph and cody).
---
 .eslintrc.js | 24 +-----------------------
 1 file changed, 1 insertion(+), 23 deletions(-)

diff --git a/.eslintrc.js b/.eslintrc.js
index cd3c2191..955b350e 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -257,29 +257,7 @@ module.exports = {
     'unicorn/prefer-number-properties': 'off',
     'unicorn/custom-error-definition': 'off', // false positives: https://github.com/sindresorhus/eslint-plugin-unicorn/issues/753
     'unicorn/no-nested-ternary': 'off', // if-elseif-else ternaries are commonly needed in JSX and formatted well by Prettier
-    'id-length': [
-      'error',
-      {
-        min: 3,
-        properties: 'never',
-        exceptions: [
-          // valid
-          'to',
-          'as',
-          'id',
-          'it', // BDD testing
-          // NodeJS standard library
-          'fs',
-          'os',
-          // conventionally used for import * as H from 'history' to not conflict with the global history
-          'H',
-          // allow `distinctUntilChanged((a, b) => isEqual(a, b))`,
-          // which is extremely common and necessary to maintain type safety.
-          'a',
-          'b',
-        ],
-      },
-    ],
+    'id-length': 'off',
     'unicorn/prevent-abbreviations': 'off',
     'unused-imports/no-unused-imports': 'error',
     'unused-imports/no-unused-vars': 'off',