build: Switch @sentry/gatsby to using eslint

Author: AbhiPrasad

.eslintignore

@@ -2,7 +2,6 @@
 # THIS WILL BE REMOVED AFTER WE FINISH ESLINT UPGRADE
 
 packages/ember/**/*
-packages/gatsby/**/*
 packages/integrations/**/*
 packages/node/**/*
 packages/react/**/*

packages/angular/.eslintrc.js

@@ -2,6 +2,7 @@ module.exports = {
   root: true,
   env: {
     es6: true,
+    browser: true,
   },
   parserOptions: {
     ecmaVersion: 2018,

packages/gatsby/.eslintrc.js

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

packages/gatsby/gatsby-browser.js

@@ -6,11 +6,15 @@ exports.onClientEntry = function(_, pluginParams) {
     let BrowserTracingIntegration = undefined;
     try {
       BrowserTracingIntegration = require('@sentry/tracing').Integrations.BrowserTracing;
-    } catch (_) {}
+    } catch (_) {
+      /* no-empty */
+    }
     try {
       /** @deprecated Remove when @sentry/apm is no longer used */
       TracingIntegration = require('@sentry/apm').Integrations.Tracing;
-    } catch (_) {}
+    } catch (_) {
+      /* no-empty */
+    }
 
     const tracesSampleRate = pluginParams.tracesSampleRate !== undefined ? pluginParams.tracesSampleRate : 0;
     const integrations = [...(pluginParams.integrations || [])];
@@ -25,7 +29,9 @@ exports.onClientEntry = function(_, pluginParams) {
 
     Sentry.init({
       environment: process.env.NODE_ENV || 'development',
+      // eslint-disable-next-line no-undef
       release: __SENTRY_RELEASE__,
+      // eslint-disable-next-line no-undef
       dsn: __SENTRY_DSN__,
       ...pluginParams,
       tracesSampleRate,

packages/gatsby/gatsby-node.js

@@ -3,7 +3,7 @@ exports.onCreateWebpackConfig = ({ plugins, actions }) => {
     plugins: [
       plugins.define({
         __SENTRY_RELEASE__: JSON.stringify(
-            process.env.SENTRY_RELEASE ||
+          process.env.SENTRY_RELEASE ||
             // GitHub Actions - https://help.github.com/en/actions/configuring-and-managing-workflows/using-environment-variables#default-environment-variables
             process.env.GITHUB_SHA ||
             // Netlify - https://docs.netlify.com/configure-builds/environment-variables/#build-metadata

packages/gatsby/package.json

@@ -50,13 +50,12 @@
     "build:watch:esm": "tsc -p tsconfig.esm.json -w --preserveWatchOutput",
     "clean": "rimraf dist coverage build esm",
     "link:yarn": "yarn link",
-    "lint": "run-s lint:prettier lint:tslint",
-    "lint:prettier": "prettier-check \"{src,test}/**/*.{ts,tsx}\"",
-    "lint:tslint": "tslint -t stylish -p .",
-    "lint:tslint:json": "tslint --format json -p . | tee lint-results.json",
-    "fix": "run-s fix:tslint fix:prettier",
-    "fix:prettier": "prettier --write \"{src,test}/**/*.{ts,tsx}\"",
-    "fix:tslint": "tslint --fix -t stylish -p .",
+    "lint": "run-s lint:prettier lint:eslint",
+    "lint:prettier": "prettier-check \"{src,test}/**/*.ts\"",
+    "lint:eslint": "eslint . --cache --cache-location '../../eslintcache/' --format stylish",
+    "fix": "run-s fix:eslint fix:prettier",
+    "fix:prettier": "prettier --write \"{src,test}/**/*.ts\"",
+    "fix:eslint": "eslint . --format stylish --fix",
     "test": "jest",
     "test:watch": "jest --watch"
   },