ref(eslint): Consolidate initial set of packages (#4102)

Convert angular, core, gatsby, hub, integrations, minimal, tracing, types, utils, vue, wasm. Certain packages (browser, react, nextjs, serverless, node) skipped as they are more involved.

Co-authored-by: iker barriocanal <[email protected]>

Author: AbhiPrasad

.eslintrc.js

@@ -0,0 +1,27 @@
+module.exports = {
+  root: true,
+  env: {
+    es6: true,
+  },
+  parserOptions: {
+    ecmaVersion: 2018,
+  },
+  extends: ['@sentry-internal/sdk'],
+  ignorePatterns: [
+    'coverage/**',
+    'build/**',
+    'dist/**',
+    'esm/**',
+    'examples/**',
+    'scripts/**',
+    'test/manual/**',
+  ],
+  overrides: [
+    {
+      files: ['*.ts', '*.tsx', '*.d.ts'],
+      parserOptions: {
+        project: './tsconfig.json',
+      },
+    }
+  ],
+};

packages/angular/.eslintrc.js

@@ -1,20 +1,6 @@
 module.exports = {
-  root: true,
   env: {
-    es6: true,
     browser: true,
   },
-  parserOptions: {
-    ecmaVersion: 2018,
-  },
-  extends: ['@sentry-internal/sdk'],
-  ignorePatterns: ['build/**', 'dist/**', 'esm/**', 'examples/**', 'scripts/**'],
-  overrides: [
-    {
-      files: ['*.ts', '*.tsx', '*.d.ts'],
-      parserOptions: {
-        project: './tsconfig.json',
-      },
-    },
-  ],
+  extends: ['../../.eslintrc.js'],
 };

packages/angular/package.json

@@ -31,7 +31,6 @@
     "@angular/common": "^10.0.3",
     "@angular/core": "^10.0.3",
     "@angular/router": "^10.0.3",
-    "@sentry-internal/eslint-config-sdk": "6.13.3",
     "npm-run-all": "^4.1.2",
     "prettier": "1.19.0",
     "rimraf": "^2.6.3",

packages/core/.eslintrc.js

@@ -1,26 +1,3 @@
 module.exports = {
-  root: true,
-  env: {
-    es6: true,
-  },
-  parserOptions: {
-    ecmaVersion: 2018,
-  },
-  extends: ['@sentry-internal/sdk'],
-  ignorePatterns: ['build/**', 'dist/**', 'esm/**', 'examples/**', 'scripts/**'],
-  overrides: [
-    {
-      files: ['*.ts', '*.tsx', '*.d.ts'],
-      parserOptions: {
-        project: './tsconfig.json',
-      },
-    },
-    {
-      files: ['test/**'],
-      rules: {
-        '@typescript-eslint/no-explicit-any': 'off',
-        '@typescript-eslint/no-non-null-assertion': 'off',
-      },
-    },
-  ],
+  extends: ['../../.eslintrc.js'],
 };

packages/core/package.json

@@ -23,7 +23,6 @@
     "tslib": "^1.9.3"
   },
   "devDependencies": {
-    "@sentry-internal/eslint-config-sdk": "6.13.3",
     "jest": "^24.7.1",
     "npm-run-all": "^4.1.2",
     "prettier": "1.19.0",

packages/eslint-config-sdk/src/index.js

@@ -163,15 +163,16 @@ module.exports = {
       env: {
         jest: true,
       },
-      files: ['*.test.ts', '*.test.tsx', '*.test.js', '*.test.jsx'],
+      files: ['*.test.ts', '*.test.tsx', '*.test.js', '*.test.jsx', 'test/**/*.ts', 'test/**/*.js'],
       rules: {
         'max-lines': 'off',
-
         '@typescript-eslint/explicit-function-return-type': 'off',
         'no-unused-expressions': 'off',
         '@typescript-eslint/no-unused-expressions': 'off',
         '@typescript-eslint/no-unsafe-member-access': 'off',
         '@typescript-eslint/explicit-member-accessibility': 'off',
+        '@typescript-eslint/no-explicit-any': 'off',
+        '@typescript-eslint/no-non-null-assertion': 'off',
       },
     },
     {

packages/gatsby/.eslintrc.js

@@ -1,22 +1,10 @@
 module.exports = {
-  root: true,
   env: {
-    es6: true,
     browser: true,
     node: true,
   },
   parserOptions: {
-    ecmaVersion: 2018,
     jsx: true,
   },
-  extends: ['@sentry-internal/sdk'],
-  ignorePatterns: ['build/**', 'dist/**', 'esm/**', 'examples/**', 'scripts/**'],
-  overrides: [
-    {
-      files: ['*.ts', '*.tsx', '*.d.ts'],
-      parserOptions: {
-        project: './tsconfig.json',
-      },
-    },
-  ],
+  extends: ['../../.eslintrc.js'],
 };

packages/gatsby/package.json

@@ -33,7 +33,6 @@
     "gatsby": "^2.0.0 || ^3.0.0"
   },
   "devDependencies": {
-    "@sentry-internal/eslint-config-sdk": "6.13.3",
     "@sentry/types": "6.13.3",
     "@testing-library/react": "^10.4.9",
     "jest": "^24.7.1",

packages/hub/.eslintrc.js

@@ -1,26 +1,3 @@
 module.exports = {
-  root: true,
-  env: {
-    es6: true,
-  },
-  parserOptions: {
-    ecmaVersion: 2018,
-  },
-  extends: ['@sentry-internal/sdk'],
-  ignorePatterns: ['build/**', 'dist/**', 'esm/**', 'examples/**', 'scripts/**'],
-  overrides: [
-    {
-      files: ['*.ts', '*.tsx', '*.d.ts'],
-      parserOptions: {
-        project: './tsconfig.json',
-      },
-    },
-    {
-      files: ['test/**'],
-      rules: {
-        '@typescript-eslint/no-explicit-any': 'off',
-        '@typescript-eslint/no-non-null-assertion': 'off',
-      },
-    },
-  ],
+  extends: ['../../.eslintrc.js'],
 };

packages/hub/package.json

@@ -21,7 +21,6 @@
     "tslib": "^1.9.3"
   },
   "devDependencies": {
-    "@sentry-internal/eslint-config-sdk": "6.13.3",
     "jest": "^24.7.1",
     "npm-run-all": "^4.1.2",
     "prettier": "1.19.0",

packages/integrations/.eslintrc.js

@@ -1,29 +1,3 @@
 module.exports = {
-  root: true,
-  env: {
-    es6: true,
-  },
-  parserOptions: {
-    ecmaVersion: 2018,
-  },
-  extends: ['@sentry-internal/sdk'],
-  ignorePatterns: ['build/**', 'dist/**', 'esm/**', 'examples/**', 'scripts/**'],
-  overrides: [
-    {
-      files: ['*.ts', '*.tsx', '*.d.ts'],
-      parserOptions: {
-        project: './tsconfig.json',
-      },
-    },
-    {
-      files: ['test/**'],
-      env: {
-        mocha: true,
-      },
-      rules: {
-        '@typescript-eslint/no-explicit-any': 'off',
-        '@typescript-eslint/no-non-null-assertion': 'off',
-      },
-    },
-  ],
+  extends: ['../../.eslintrc.js'],
 };

packages/integrations/package.json

@@ -22,7 +22,6 @@
     "tslib": "^1.9.3"
   },
   "devDependencies": {
-    "@sentry-internal/eslint-config-sdk": "6.13.3",
     "chai": "^4.1.2",
     "jest": "^24.7.1",
     "npm-run-all": "^4.1.2",

packages/minimal/.eslintrc.js

@@ -1,26 +1,3 @@
 module.exports = {
-  root: true,
-  env: {
-    es6: true,
-  },
-  parserOptions: {
-    ecmaVersion: 2018,
-  },
-  extends: ['@sentry-internal/sdk'],
-  ignorePatterns: ['build/**', 'dist/**', 'esm/**', 'examples/**', 'scripts/**'],
-  overrides: [
-    {
-      files: ['*.ts', '*.tsx', '*.d.ts'],
-      parserOptions: {
-        project: './tsconfig.json',
-      },
-    },
-    {
-      files: ['test/**'],
-      rules: {
-        '@typescript-eslint/no-explicit-any': 'off',
-        '@typescript-eslint/no-non-null-assertion': 'off',
-      },
-    },
-  ],
+  extends: ['../../.eslintrc.js'],
 };

packages/minimal/package.json

@@ -21,7 +21,6 @@
     "tslib": "^1.9.3"
   },
   "devDependencies": {
-    "@sentry-internal/eslint-config-sdk": "6.13.3",
     "jest": "^24.7.1",
     "npm-run-all": "^4.1.2",
     "prettier": "1.19.0",

packages/node/.eslintrc.js

@@ -1,7 +1,6 @@
 module.exports = {
   root: true,
   env: {
-    es6: true,
     node: true,
   },
   parserOptions: {

packages/react/test/errorboundary.test.tsx

@@ -1,5 +1,4 @@
 import { Scope } from '@sentry/browser';
-import { Event, Severity } from '@sentry/types';
 import { fireEvent, render, screen } from '@testing-library/react';
 import * as React from 'react';
 import { useState } from 'react';

packages/tracing/.eslintrc.js

@@ -1,26 +1,3 @@
 module.exports = {
-  root: true,
-  env: {
-    es6: true,
-  },
-  parserOptions: {
-    ecmaVersion: 2018,
-  },
-  extends: ['@sentry-internal/sdk'],
-  ignorePatterns: ['build/**', 'dist/**', 'esm/**', 'examples/**', 'scripts/**'],
-  overrides: [
-    {
-      files: ['*.ts', '*.tsx', '*.d.ts'],
-      parserOptions: {
-        project: './tsconfig.json',
-      },
-    },
-    {
-      files: ['test/**'],
-      rules: {
-        '@typescript-eslint/no-explicit-any': 'off',
-        '@typescript-eslint/no-non-null-assertion': 'off',
-      },
-    },
-  ],
+  extends: ['../../.eslintrc.js'],
 };

packages/tracing/package.json

@@ -23,7 +23,6 @@
     "tslib": "^1.9.3"
   },
   "devDependencies": {
-    "@sentry-internal/eslint-config-sdk": "6.13.3",
     "@sentry/browser": "6.13.3",
     "@types/express": "^4.17.1",
     "@types/jsdom": "^16.2.3",

packages/types/.eslintrc.js

@@ -1,22 +1,6 @@
 module.exports = {
-  root: true,
-  env: {
-    es6: true,
-  },
-  parserOptions: {
-    ecmaVersion: 2018,
-  },
-  extends: ['@sentry-internal/sdk'],
-  ignorePatterns: ['build/**', 'dist/**', 'esm/**', 'examples/**', 'scripts/**'],
-  overrides: [
-    {
-      files: ['*.ts', '*.tsx', '*.d.ts'],
-      parserOptions: {
-        project: './tsconfig.json',
-      },
-    },
-  ],
+  extends: ['../../.eslintrc.js'],
   rules: {
     '@typescript-eslint/no-explicit-any': 'off',
-  },
+  }
 };

packages/types/package.json

@@ -16,7 +16,6 @@
     "access": "public"
   },
   "devDependencies": {
-    "@sentry-internal/eslint-config-sdk": "6.13.3",
     "npm-run-all": "^4.1.2",
     "prettier": "1.19.0",
     "typescript": "3.7.5"

packages/utils/.eslintrc.js

@@ -1,29 +1,3 @@
 module.exports = {
-  root: true,
-  env: {
-    es6: true,
-  },
-  parserOptions: {
-    ecmaVersion: 2018,
-  },
-  extends: ['@sentry-internal/sdk'],
-  ignorePatterns: ['build/**', 'dist/**', 'esm/**', 'examples/**', 'scripts/**'],
-  overrides: [
-    {
-      files: ['*.ts', '*.tsx', '*.d.ts'],
-      parserOptions: {
-        project: './tsconfig.json',
-      },
-    },
-    {
-      files: ['test/**'],
-      rules: {
-        '@typescript-eslint/no-explicit-any': 'off',
-        '@typescript-eslint/no-non-null-assertion': 'off',
-      },
-    },
-  ],
-  rules: {
-    'max-lines': 'off',
-  },
+  extends: ['../../.eslintrc.js'],
 };

packages/utils/src/instrument.ts

@@ -1,3 +1,4 @@
+/* eslint-disable max-lines */
 /* eslint-disable @typescript-eslint/no-explicit-any */
 /* eslint-disable @typescript-eslint/ban-types */
 import { WrappedFunction } from '@sentry/types';

packages/utils/src/object.ts

@@ -1,3 +1,4 @@
+/* eslint-disable max-lines */
 /* eslint-disable @typescript-eslint/no-explicit-any */
 import { ExtendedError, WrappedFunction } from '@sentry/types';