build: Remove tslint danger

Author: AbhiPrasad

.eslintignore

@@ -1,7 +1,6 @@
 # THIS IS A TEMPORARY FILE
 # THIS WILL BE REMOVED AFTER WE FINISH ESLINT UPGRADE
 
-packages/node/**/*
 packages/react/**/*
 packages/tracing/**/*
 packages/typescript/**/*

.eslintrc.js

@@ -4,8 +4,8 @@ module.exports = {
     node: true,
   },
   extends: ['prettier', 'eslint:recommended', 'plugin:import/errors', 'plugin:import/warnings'],
-  plugins: ['sentry-sdk', 'simple-import-sort'],
-  ignorePatterns: ['eslint-plugin-sentry-sdk'],
+  plugins: ['sentry-sdks', 'simple-import-sort'],
+  ignorePatterns: ['eslint-plugin-sentry-sdks'],
   overrides: [
     {
       // Configuration for JavaScript files

dangerfile.ts

@@ -1,12 +1,8 @@
 import { exec } from 'child_process';
 import { danger, fail, message, schedule, warn } from 'danger';
-import tslint from 'danger-plugin-tslint';
-import { prettyResults } from 'danger-plugin-tslint/dist/prettyResults';
 import { CLIEngine } from 'eslint';
-import { resolve } from 'path';
 import { promisify } from 'util';
 
-const PACKAGES = ['node'];
 const EXTENSIONS = ['.js', '.jsx', '.ts', '.tsx'];
 
 /**
@@ -48,35 +44,6 @@ export default async (): Promise<void> => {
     return;
   }
 
-  schedule(async () => {
-    const tsLintResult = (
-      await Promise.all(
-        PACKAGES.map(packageName => {
-          return new Promise<string>(res => {
-            tslint({
-              lintResultsJsonPath: resolve(__dirname, 'packages', packageName, 'lint-results.json'),
-              handleResults: results => {
-                if (results.length > 0) {
-                  const formattedResults = prettyResults(results);
-                  res(`TSLint failed: **@sentry/${packageName}**\n\n${formattedResults}`);
-                } else {
-                  res('');
-                }
-              },
-            });
-          });
-        }),
-      )
-    ).filter(str => str.length);
-    if (tsLintResult.length) {
-      tsLintResult.forEach(tsLintFail => {
-        fail(`${tsLintFail}`);
-      });
-    } else {
-      message('✅ TSLint passed');
-    }
-  });
-
   await eslint();
 
   const hasChangelog = danger.git.modified_files.indexOf('CHANGELOG.md') !== -1;

eslint-plugin-sentry-sdk/index.js renamed to eslint-plugin-sentry-sdks/index.js

@@ -1,5 +1,5 @@
 {
-  "name": "eslint-plugin-sentry-sdk",
+  "name": "eslint-plugin-sentry-sdks",
   "version": "0.0.1",
   "main": "index.js",
   "devDependencies": {

eslint-plugin-sentry-sdk/package.json renamed to eslint-plugin-sentry-sdks/package.json

@@ -49,14 +49,12 @@
     "chai": "^4.1.2",
     "codecov": "^3.6.5",
     "danger": "^7.1.3",
-    "danger-plugin-eslint": "^0.1.0",
-    "danger-plugin-tslint": "^2.0.0",
     "eslint": "^7.5.0",
     "eslint-config-prettier": "^6.11.0",
     "eslint-plugin-deprecation": "^1.1.0",
     "eslint-plugin-import": "^2.22.0",
     "eslint-plugin-jsdoc": "^30.0.3",
-    "eslint-plugin-sentry-sdk": "file:./eslint-plugin-sentry-sdk",
+    "eslint-plugin-sentry-sdks": "file:./eslint-plugin-sentry-sdks",
     "eslint-plugin-simple-import-sort": "^5.0.3",
     "jest": "^24.7.1",
     "karma-browserstack-launcher": "^1.5.1",