github / github/eslint-plugin-github
Request: Add typescript support
- Vorherrschende Sprache
- JavaScript
- Sterne
- 336
- Forks
- 67
- Ø Merge
- 1 T. 9 Std.
- Gemergte PRs (30 T.)
- 6
Beschreibung
I'm working on an addition to [insights-web](https://github.com/github/insights-web) ([PR](https://github.com/github/insights-web/pull/123)) 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 configuring [eslint-import-resolver-typescript](https://www.npmjs.com/package/eslint-import-resolver-typescript) to the project. I wanted to submit this issue here though because this seems like a better addition to `estlint-plugin-github`, that way we have org-wide TS linting support. There's also an alternative [eslint-plugin-i](https://github.com/un-es/eslint-plugin-i) package that claims to be faster than `eslint-plugin-import`.
Beitragsleitfaden
Rechercherichtung
Überprüfe die aktuelle Dependency- und Resolver-Konfiguration von eslint-plugin-github und vergleiche anschließend den vorgeschlagenen Ansatz mit eslint-import-resolver-typescript mit der alternativen eslint-plugin-i-Paket. Als abgeschlossen gilt die Aufgabe, wenn TypeScript-.d.ts-Imports unter den ESLint-Regeln des Plugins korrekt aufgelöst werden und die bestehenden Lint-Prüfungen erfolgreich durchlaufen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript, typescript
- Bereich
- tooling
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100