microsoft / microsoft/TypeScript

noUnusedParameters incorrectly triggers with @interface JSDoc in JS files

Offen
#43,791 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Awaiting More Feedback Suggestion
Vorherrschende Sprache
Go
Sterne
111k
Forks
14.3k
Ø Merge
1 T. 19 Std.
Gemergte PRs (30 T.)
117

Beschreibung

Bug Report

🔎 Search Terms

noUnusedParameters @interface

🕗 Version & Regression Information

Playground link with relevant code

💻 Code
/** @interface */
class FooInterface {
  /** @param {string} id */
  doSomething(id) {}
}
🙁 Actual behavior

Throws the following error

'id' is declared but its value is never read.
🙂 Expected behavior

There should be no error thrown, as this is the correct (and only?) way to declare interfaces with JSDoc in JavaScript files (and also with Closure Compiler)

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Reproduziere die Diagnose im verlinkten TypeScript Playground mit aktiviertem noUnusedParameters und ausgewähltem JavaScript-Modus. Verfolge, wie @interface JSDoc und der Parameter doSomething(id) behandelt werden, bestätige anschließend, dass das gültige Interface-Muster 'id' nicht mehr als unbenutzt meldet, und füge eine Regressionstestabdeckung für diesen Fall hinzu.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript, typescript
Bereich
compilers
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

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