- Notifications
You must be signed in to change notification settings - Fork61
Open
Description
I'm working on an addition toinsights-web (PR) that requires an upgrade to Chart.js 4. Chart.js 4 added typescript.d.ts
definitions for various exported types. This breaks the GitHub eslint rules because the linter isn't able to resolve the imported types, despite everything compiling correctly. I was able to fix the linting by adding and configuringeslint-import-resolver-typescript to the project. I wanted to submit this issue here though because this seems like a better addition toestlint-plugin-github
, that way we have org-wide TS linting support. There's also an alternativeeslint-plugin-i package that claims to be faster thaneslint-plugin-import
.
Metadata
Metadata
Assignees
Labels
No labels