Skip to content

Conversation

DonIsaac
Copy link
Contributor

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

@bmschmidt
Copy link
Collaborator

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

@DonIsaac
Copy link
Contributor Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants