Merge pull request #1754 from skaut/disable-naming-convention-rule

Disabled naming-convention eslint rule

Author: marekdedic

packages/collector/__tests__/setup.ts

@@ -6,7 +6,6 @@ import nodeFetch from "node-fetch";
 nock.disableNetConnect();
 
 jest.mock("../src/octokit", () => {
-  // eslint-disable-next-line @typescript-eslint/naming-convention -- Classes should be PascalCase
   const OctokitWithRest = Octokit.plugin(restEndpointMethods);
   return {
     octokit: new OctokitWithRest({

packages/collector/eslint.config.js

@@ -7,8 +7,6 @@ import preferArrowFunctions from "eslint-plugin-prefer-arrow-functions";
 import prettierRecommended from "eslint-plugin-prettier/recommended";
 import tseslint from "typescript-eslint";
 
-/* eslint-disable @typescript-eslint/naming-convention -- Not applicable to this file */
-
 export default tseslint.config(
   js.configs.recommended,
   prettierRecommended,
@@ -41,33 +39,6 @@ export default tseslint.config(
       "@typescript-eslint/explicit-module-boundary-types": "error",
       "@typescript-eslint/init-declarations": "error",
       "@typescript-eslint/method-signature-style": ["error", "method"],
-      "@typescript-eslint/naming-convention": [
-        "error",
-        {
-          filter: {
-            match: false,
-            regex: "^(help-issue-label|html_url|per_page|short-description)$",
-          },
-          format: ["camelCase"],
-          leadingUnderscore: "allow",
-          selector: "default",
-          trailingUnderscore: "allow",
-        },
-        {
-          format: ["camelCase", "PascalCase"],
-          selector: "import",
-        },
-        {
-          format: ["camelCase", "UPPER_CASE"],
-          leadingUnderscore: "allow",
-          selector: "variable",
-          trailingUnderscore: "allow",
-        },
-        {
-          format: ["PascalCase"],
-          selector: "typeLike",
-        },
-      ],
       "@typescript-eslint/no-import-type-side-effects": "error",
       "@typescript-eslint/no-shadow": "error",
       "@typescript-eslint/no-unnecessary-parameter-property-assignment":
@@ -218,5 +189,3 @@ export default tseslint.config(
     },
   },
 );
-
-/* eslint-enable @typescript-eslint/naming-convention */

packages/collector/jest.config.js

@@ -14,7 +14,6 @@ export default {
   setupFiles: ["<rootDir>/__tests__/setup.ts"],
   testMatch: ["<rootDir>/__tests__/**/*.test.ts"],
   transform: {
-    // eslint-disable-next-line @typescript-eslint/naming-convention -- Regexp key
     "^.+\\.[jt]s$": [
       "ts-jest",
       {

packages/collector/src/octokit.ts

@@ -1,6 +1,5 @@
 import { Octokit } from "@octokit/core";
 import { restEndpointMethods } from "@octokit/plugin-rest-endpoint-methods";
 
-// eslint-disable-next-line @typescript-eslint/naming-convention -- Classes should be PascalCase
 const OctokitWithRest = Octokit.plugin(restEndpointMethods);
 export const octokit = new OctokitWithRest({ auth: process.env["PAT"] });

packages/frontend/eslint.config.js

@@ -10,8 +10,6 @@ import react from "eslint-plugin-react";
 import globals from "globals";
 import tseslint from "typescript-eslint";
 
-/* eslint-disable @typescript-eslint/naming-convention -- Not applicable to this file */
-
 export default tseslint.config(
   js.configs.recommended,
   prettierRecommended,
@@ -47,33 +45,6 @@ export default tseslint.config(
       "@typescript-eslint/explicit-module-boundary-types": "error",
       "@typescript-eslint/init-declarations": "error",
       "@typescript-eslint/method-signature-style": ["error", "method"],
-      "@typescript-eslint/naming-convention": [
-        "error",
-        {
-          filter: {
-            match: false,
-            regex: "^(help-issue-label|short-description)$",
-          },
-          format: ["camelCase"],
-          leadingUnderscore: "allow",
-          selector: "default",
-          trailingUnderscore: "allow",
-        },
-        {
-          format: ["camelCase", "PascalCase"],
-          selector: "import",
-        },
-        {
-          format: ["camelCase", "PascalCase", "UPPER_CASE"],
-          leadingUnderscore: "allow",
-          selector: "variable",
-          trailingUnderscore: "allow",
-        },
-        {
-          format: ["PascalCase"],
-          selector: "typeLike",
-        },
-      ],
       "@typescript-eslint/no-import-type-side-effects": "error",
       "@typescript-eslint/no-shadow": "error",
       "@typescript-eslint/no-unnecessary-parameter-property-assignment":
@@ -296,5 +267,3 @@ export default tseslint.config(
     },
   },
 );
-
-/* eslint-enable @typescript-eslint/naming-convention */

packages/frontend/jest.config.js

@@ -8,15 +8,13 @@ export default {
     "!src/globalStyles.ts",
   ],
   moduleNameMapper: {
-    // eslint-disable-next-line @typescript-eslint/naming-convention -- The key is a glob matching files
     "\\.svg$": "<rootDir>/__mocks__/fileMock.ts",
   },
   resetMocks: true,
   setupFilesAfterEnv: ["<rootDir>/__tests__/setupTests.ts"],
   testEnvironment: "jsdom",
   testMatch: ["<rootDir>/__tests__/**/*.test.{js,jsx,ts,tsx}"],
   transform: {
-    // eslint-disable-next-line @typescript-eslint/naming-convention -- The key is a glob matching files
     "^.+\\.(j|t)sx?$": ["ts-jest", { tsconfig: "test.tsconfig.json" }],
   },
   transformIgnorePatterns: [