deepscatter icon indicating copy to clipboard operation
deepscatter copied to clipboard

Allow eslint-plugin-import to Resolve TypeScript Files

Open DonIsaac opened this issue 2 years ago • 2 comments

What This PR Does

  • Allow JS files to use CommonJS require()
  • Add eslint-import-resolver-typescript so that eslint-plugin-import can resolve TypeScript imports
  • Add typescript as a dev dependency
  • Add a npm run typecheck script to run tsc on source code without emitting compiled output

DonIsaac avatar Aug 23 '22 17:08 DonIsaac

Is require() desirable? As someone who avoided javascript in the bad old days, I have an aversion.

bmschmidt avatar Aug 23 '22 19:08 bmschmidt

No, but it's needed in config files. I can modify the rules to only apply to those

DonIsaac avatar Aug 23 '22 20:08 DonIsaac