build: Switch minimal to using eslint

Author: AbhiPrasad

.eslintignore

@@ -5,7 +5,6 @@ packages/apm/**/*
 packages/ember/**/*
 packages/gatsby/**/*
 packages/integrations/**/*
-packages/minimal/**/*
 packages/node/**/*
 packages/react/**/*
 packages/tracing/**/*

packages/minimal/.eslintrc.js

@@ -0,0 +1,26 @@
+module.exports = {
+  root: true,
+  env: {
+    es6: true,
+  },
+  parserOptions: {
+    ecmaVersion: 2018,
+  },
+  extends: ['../../.eslintrc.js'],
+  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',
+      },
+    },
+  ],
+};

packages/minimal/package.json

@@ -26,7 +26,6 @@
     "prettier": "^1.17.0",
     "prettier-check": "^2.0.0",
     "rimraf": "^2.6.3",
-    "tslint": "5.16.0",
     "typescript": "3.4.5"
   },
   "scripts": {
@@ -38,13 +37,13 @@
     "build:watch:esm": "tsc -p tsconfig.esm.json -w --preserveWatchOutput",
     "clean": "rimraf dist coverage",
     "link:yarn": "yarn link",
-    "lint": "run-s lint:prettier lint:tslint",
+    "lint": "run-s lint:prettier lint:eslint",
     "lint:prettier": "prettier-check \"{src,test}/**/*.ts\"",
-    "lint:tslint": "tslint -t stylish -p .",
-    "lint:tslint:json": "tslint --format json -p . | tee lint-results.json",
-    "fix": "run-s fix:tslint fix:prettier",
+    "lint:eslint": "eslint . --format stylish",
+    "lint:eslint:json": "eslint . --format json | tee lint-results.json",
+    "fix": "run-s fix:eslint fix:prettier",
     "fix:prettier": "prettier --write \"{src,test}/**/*.ts\"",
-    "fix:tslint": "tslint --fix -t stylish -p .",
+    "fix:eslint": "lint:eslint --fix",
     "test": "jest",
     "test:watch": "jest --watch"
   },

packages/minimal/src/index.ts

@@ -16,10 +16,11 @@ import {
  * @param method function to call on hub.
  * @param args to pass to function.
  */
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
 function callOnHub<T>(method: string, ...args: any[]): T {
   const hub = getCurrentHub();
   if (hub && hub[method as keyof Hub]) {
-    // tslint:disable-next-line:no-unsafe-any
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     return (hub[method as keyof Hub] as any)(...args);
   }
   throw new Error(`No hub defined or ${method} was not found on the hub, please open a bug report.`);
@@ -31,6 +32,7 @@ function callOnHub<T>(method: string, ...args: any[]): T {
  * @param exception An exception-like object.
  * @returns The generated eventId.
  */
+// eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/explicit-module-boundary-types
 export function captureException(exception: any, captureContext?: CaptureContext): string {
   let syntheticException: Error;
   try {
@@ -107,6 +109,7 @@ export function addBreadcrumb(breadcrumb: Breadcrumb): void {
  * @param name of the context
  * @param context Any kind of data. This data will be normalized.
  */
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
 export function setContext(name: string, context: { [key: string]: any } | null): void {
   callOnHub<void>('setContext', name, context);
 }
@@ -132,7 +135,6 @@ export function setTags(tags: { [key: string]: string }): void {
  * @param key String of extra
  * @param extra Any kind of data. This data will be normalized.
  */
-
 export function setExtra(key: string, extra: Extra): void {
   callOnHub<void>('setExtra', key, extra);
 }
@@ -182,6 +184,7 @@ export function withScope(callback: (scope: Scope) => void): void {
  * @param args Arguments to pass to the client/fontend.
  * @hidden
  */
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
 export function _callOnClient(method: string, ...args: any[]): void {
   callOnHub<void>('_invokeClient', method, ...args);
 }

packages/minimal/test/lib/minimal.test.ts

@@ -17,6 +17,7 @@ import {
 } from '../../src';
 import { init, TestClient, TestClient2 } from '../mocks/client';
 
+// eslint-disable-next-line no-var
 declare var global: any;
 
 describe('Minimal', () => {

packages/minimal/test/mocks/client.ts

@@ -3,7 +3,7 @@ import { getCurrentHub } from '@sentry/hub';
 export class TestClient {
   public static instance?: TestClient;
 
-  public constructor(public options: object) {
+  public constructor(public options: Record<string, unknown>) {
     TestClient.instance = this;
   }
 
@@ -14,6 +14,6 @@ export class TestClient {
 
 export class TestClient2 {}
 
-export function init(options: object): void {
+export function init(options: Record<string, unknown>): void {
   getCurrentHub().bindClient(new TestClient(options) as any);
 }

packages/minimal/test/tslint.json

@@ -1,6 +1,6 @@
 {
   "extends": "./tsconfig.build.json",
-  "include": ["src/**/*.ts"],
+  "include": ["src/**/*.ts", "test/**/*.ts"],
   "exclude": ["dist"],
   "compilerOptions": {
     "rootDir": ".",