Add mixed mode for consistent-type-imports rule

This ads a new mode called "" for consistent-type-imports rule which will only trigger, if all imports of an import declaration are type only imports. References typescript-eslint#2769
olee · Dec 5, 2020 · a491fd7 · a491fd7
1 parent ccb6b94
commit a491fd7
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/eslint-plugin/src/rules/consistent-type-imports.ts b/packages/eslint-plugin/src/rules/consistent-type-imports.ts
@@ -6,7 +6,7 @@ import {
 } from '@typescript-eslint/experimental-utils';
 import * as util from '../util';
 
-type Prefer = 'type-imports' | 'no-type-imports';
+type Prefer = 'type-imports' | 'no-type-imports' | 'type-imports-mixed';
 
 type Options = [
   {
@@ -94,7 +94,7 @@ export default util.createRule<Options, MessageIds>({
     const sourceImportsMap: { [key: string]: SourceImports } = {};
 
     return {
-      ...(prefer === 'type-imports'
+      ...(prefer !== 'no-type-imports'
         ? {
             // prefer type imports
             ImportDeclaration(node: TSESTree.ImportDeclaration): void {
@@ -157,7 +157,7 @@ export default util.createRule<Options, MessageIds>({
                 }
               }
 
-              if (typeSpecifiers.length) {
+              if (typeSpecifiers.length && (prefer === 'type-imports' || valueSpecifiers.length === 0)) {
                 sourceImports.reportValueImports.push({
                   node,
                   typeSpecifiers,