Aug 8, 2020 · Aug 8, 2020
diff --git a/packages/eslint-plugin/src/rules/no-shadow.ts b/packages/eslint-plugin/src/rules/no-shadow.ts
      );
    }

    /**
     * Checks if a variable of the class name in the class scope of ClassDeclaration.
     *
     * ClassDeclaration creates two variables of its name into its outer scope and its class scope.
     * So we should ignore the variable in the class scope.
     * @param variable The variable to check.
     * @returns Whether or not the variable of the class name in the class scope of ClassDeclaration.
     */
    function isDuplicatedEnumNameVariable(
      variable: TSESLint.Scope.Variable,
    ): boolean {
      const block = variable.scope.block;

      return (
        block.type === AST_NODE_TYPES.TSEnumDeclaration &&
        block.id === variable.identifiers[0]
      );
    }

    /**
     * Checks if a variable is inside the initializer of scopeVar.
     *
          continue;
        }

        // ignore variables of a class name in the class scope of ClassDeclaration
        if (isDuplicatedEnumNameVariable(variable)) {
          continue;
        }

        // ignore configured allowed names
        if (isAllowed(variable)) {
          continue;
diff --git a/packages/eslint-plugin/tests/rules/no-shadow.test.ts b/packages/eslint-plugin/tests/rules/no-shadow.test.ts
      `,
      options: [{ allow: ['cb'] }],
    },
    // https://github.com/typescript-eslint/typescript-eslint/issues/2360
    `
 enum Direction {
  left = 'left',
  right = 'right',
 }
    `,
  ],
  invalid: [
    {
Original file line number	Diff line number	Diff line change
Expand Up		@@ -116,6 +116,25 @@ export default util.createRule<Options, MessageIds>({
		);
		}

		/**
		* Checks if a variable of the class name in the class scope of ClassDeclaration.
		*
		* ClassDeclaration creates two variables of its name into its outer scope and its class scope.
		* So we should ignore the variable in the class scope.
		* @param variable The variable to check.
		* @returns Whether or not the variable of the class name in the class scope of ClassDeclaration.
		*/
		function isDuplicatedEnumNameVariable(
		variable: TSESLint.Scope.Variable,
		): boolean {
		const block = variable.scope.block;

		return (
		block.type === AST_NODE_TYPES.TSEnumDeclaration &&
		block.id === variable.identifiers[0]
		);
		}

		/**
		* Checks if a variable is inside the initializer of scopeVar.
		*
Expand DownExpand Up		@@ -233,6 +252,11 @@ export default util.createRule<Options, MessageIds>({
		continue;
		}

		// ignore variables of a class name in the class scope of ClassDeclaration
		if (isDuplicatedEnumNameVariable(variable)) {
		continue;
		}

		// ignore configured allowed names
		if (isAllowed(variable)) {
		continue;
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -431,6 +431,13 @@ function foo(cb) {
		`,
		options: [{ allow: ['cb'] }],
		},
		// https://github.com/typescript-eslint/typescript-eslint/issues/2360
		`
		enum Direction {
		left = 'left',
		right = 'right',
		}
		`,
		],
		invalid: [
		{
Expand Down