github / github/eslint-plugin-github

Request: Add typescript support

Offen
#406 1 Kommentar 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen
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

Beitragsleitfaden öffnen

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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.