From 59586f8be6c0ccfcfe16a0f1f9fbb73473e1bbbf Mon Sep 17 00:00:00 2001
From: Gustav Tiger <gustav.tiger@axis.com>
Date: Thu, 1 Oct 2020 20:01:54 +0200
Subject: [PATCH] feat: Use @typescript-eslint version of
 no-shadow/no-redeclare

Use @typescript-eslint versions of the no-shadow and no-redeclare rules.

Remove the no-undef rule since that is covered by typescript and not in
the recommended set of rules. See also
<https://github.com/typescript-eslint/typescript-eslint/issues/2477#issuecomment-686892459>.
---
 index.js | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/index.js b/index.js
index 9d62376..27d3330 100644
--- a/index.js
+++ b/index.js
@@ -122,8 +122,6 @@ module.exports = {
     'no-new-wrappers': 'error',
     /** Disallow octal literals */
     'no-octal': 'error',
-    /** Disallow variable redeclaration */
-    'no-redeclare': 'error',
     /** Disallow assignments where both sides are exactly the same */
     'no-self-assign': 'error',
     /** Disallow `javascript:` urls */
@@ -140,12 +138,8 @@ module.exports = {
     /**
      * Variables
      */
-    /** Disallow variable declarations from shadowing variables declared in the outer scope */
-    'no-shadow': 'error',
     /** Disallow identifiers from shadowing restricted names */
     'no-shadow-restricted-names': 'error',
-    /** disallow the use of undeclared variables unless mentioned in global comments */
-    'no-undef': 'error',
 
     /**
      * ECMAScript 6
@@ -222,8 +216,12 @@ module.exports = {
     '@typescript-eslint/no-non-null-assertion': 'error',
     /** Disallow the use of parameter properties in class constructors */
     '@typescript-eslint/no-parameter-properties': 'error',
+    /** Disallow variable redeclaration */
+    "@typescript-eslint/no-redeclare": "error",
     /** Disallows invocation of require() */
     '@typescript-eslint/no-require-imports': 'error',
+    /** Disallow variable declarations from shadowing variables declared in the outer scope */
+    "@typescript-eslint/no-shadow": "error",
     /** Disallow aliasing this */
     '@typescript-eslint/no-this-alias': 'error',
     /** Warns when a namespace qualifier is unnecessary */