feat: Create eslint config

Author: AbhiPrasad

.eslintignore

@@ -0,0 +1,16 @@
+# THIS IS A TEMPORARY FILE
+# THIS WILL BE REMOVED AFTER WE FINISH ESLINT UPGRADE
+
+packages/apm/**/*
+packages/core/**/*
+packages/ember/**/*
+packages/gatsby/**/*
+packages/hub/**/*
+packages/integrations/**/*
+packages/minimal/**/*
+packages/node/**/*
+packages/react/**/*
+packages/tracing/**/*
+packages/types/**/*
+packages/typescript/**/*
+packages/utils/**/*

.eslintrc.js

@@ -0,0 +1,36 @@
+module.exports = {
+  root: true,
+  env: {
+    node: true,
+  },
+  extends: ['prettier', 'eslint:recommended'],
+  plugins: ['sentry-sdk'],
+  overrides: [
+    {
+      files: ['*.ts'],
+      extends: ['plugin:@typescript-eslint/recommended', 'prettier/@typescript-eslint'],
+      plugins: ['@typescript-eslint'],
+      parser: '@typescript-eslint/parser',
+      rules: {
+        'sentry-sdk/no-async-await': 'error',
+      },
+    },
+    {
+      env: {
+        jest: true,
+      },
+      files: ['*.test.ts'],
+      extends: ['plugin:@typescript-eslint/recommended', 'prettier/@typescript-eslint'],
+      plugins: ['@typescript-eslint'],
+      parser: '@typescript-eslint/parser',
+    },
+    {
+      files: ['*.config.js'],
+      parserOptions: {
+        sourceType: 'module',
+        ecmaVersion: 2018,
+      },
+    },
+  ],
+  rules: {},
+};

.vscode/extensions.json

@@ -1,5 +1,5 @@
 {
   // See http://go.microsoft.com/fwlink/?LinkId=827846
   // for the documentation about the extensions.json format
-  "recommendations": ["esbenp.prettier-vscode", "ms-vscode.vscode-typescript-tslint-plugin"]
+  "recommendations": ["esbenp.prettier-vscode", "ms-vscode.vscode-typescript-tslint-plugin", "dbaeumer.vscode-eslint"]
 }

eslint-plugin-sentry-sdk/index.js

@@ -0,0 +1,40 @@
+// This is a temporary file. It will be removed when we migrate
+// the eslint configs to another repo.
+
+'use strict';
+
+/**
+ * Rule to disallow usage of async await
+ * @author Abhijeet Prasad
+ */
+const noAsyncAwait = {
+  meta: {
+    type: 'problem',
+    docs: {
+      description: 'Disallow usage of async await',
+      category: 'Best Practices',
+      recommended: true,
+    },
+    fixable: null,
+    schema: [],
+  },
+  create: function(context) {
+    return {
+      FunctionDeclaration(node) {
+        if (node.async) {
+          context.report({
+            node,
+            message:
+              'Using async-await can add a lot to bundle size. Please do not use it outside of tests, use Promises instead',
+          });
+        }
+      },
+    };
+  },
+};
+
+module.exports = {
+  rules: {
+    'no-async-await': noAsyncAwait,
+  },
+};

eslint-plugin-sentry-sdk/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "eslint-plugin-sentry-sdk",
+  "version": "0.0.1",
+  "main": "index.js",
+  "devDependencies": {
+    "eslint": "^7.5.0"
+  },
+  "engines": {
+    "node": ">=0.10.0"
+  }
+}

package.json

@@ -43,10 +43,15 @@
     "@types/mocha": "^5.2.0",
     "@types/node": "^11.13.7",
     "@types/sinon": "^7.0.11",
+    "@typescript-eslint/eslint-plugin": "^3.7.1",
+    "@typescript-eslint/parser": "^3.7.1",
     "chai": "^4.1.2",
     "codecov": "^3.6.5",
     "danger": "^7.1.3",
     "danger-plugin-tslint": "^2.0.0",
+    "eslint": "^7.5.0",
+    "eslint-config-prettier": "^6.11.0",
+    "eslint-plugin-sentry-sdk": "file:./eslint-plugin-sentry-sdk",
     "jest": "^24.7.1",
     "karma-browserstack-launcher": "^1.5.1",
     "karma-firefox-launcher": "^1.1.0",

packages/browser/.eslintrc.js

@@ -0,0 +1,27 @@
+module.exports = {
+  root: true,
+  env: {
+    es6: true,
+    browser: true,
+  },
+  extends: ['../../.eslintrc.js'],
+  ignorePatterns: ['build/**/*', 'dist/**/*', 'esm/**/*', 'examples/**/*', 'scripts/**/*'],
+  overrides: [
+    {
+      files: ['test/integration/**/*'],
+      env: {
+        mocha: true,
+      },
+      rules: {
+        'no-undef': 'off',
+      },
+    },
+    {
+      files: ['test/integration/common/**/*'],
+      rules: {
+        'no-unused-vars': 'off',
+      },
+    },
+  ],
+  rules: {},
+};

packages/browser/package.json

@@ -49,7 +49,6 @@
     "rollup-plugin-terser": "^4.0.4",
     "rollup-plugin-typescript2": "^0.21.0",
     "sinon": "^7.3.2",
-    "tslint": "5.16.0",
     "typescript": "3.4.5",
     "webpack": "^4.30.0"
   },
@@ -64,13 +63,13 @@
     "build:watch": "run-p build:dist:watch build:esm:watch build:bundle:watch",
     "clean": "rimraf dist coverage .rpt2_cache build esm",
     "link:yarn": "yarn link",
-    "lint": "run-s lint:prettier lint:tslint",
+    "lint": "run-s lint:eslint lint:tslint",
     "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": "run-s test:unit",
     "test:unit": "karma start test/unit/karma.conf.js",
     "test:unit:watch": "karma start test/unit/karma.conf.js --auto-watch --no-single-run",