feat: remove @typescript-eslint/consistent-type-imports

[zanminkian]

Remove @typescript-eslint/consistent-type-imports to improve compatibility for Angular and Nest.

In Ioc framework, code below are commonly used by developer.

import { AClass } from './a-class'

@Injectable()
export class BClass {
  constructor(private readonly aClass: AClass){}
}

After  formatting, code import { AClass } from './a-class' will become import type { AClass } from './a-class', which will  cause error, because typescript complier cannot get metadata by reflection.

[antfu]

Thanks but no. This one I strongly prefer to have. Feel free to override on your local.

[zanminkian]

Now (version 0.33.1), for the dependency-injection projects which, for example, base on nest or angluar, can add a tsconfig.eslint.json to the project root and make sure its compileOptions.emitDecoratorMetadata is true. Because this PR #131 works.

Check https://typescript-eslint.io/rules/consistent-type-imports/#usage-with-emitdecoratormetadata