microsoft / microsoft/TypeScript

Infer-from-usage should find references of related tokens

Offen
#28,253 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Domain: LS: Quick Fixes Experience Enhancement Suggestion
Vorherrschende Sprache
Go
Sterne
111k
Forks
14.3k
Ø Merge
2 T. 4 Std.
Gemergte PRs (30 T.)
132

Beschreibung

Based on code in acorn:

var TokenType = function TokenType(label, conf) {
  this.label = label
  this.binop = conf.binop || null
}

/**
 * @param {string} name
 * @param {number} prec
 */
function x(name, prec) {
  return new TokenType(name, { binop: prec })
}

Expected behavior:

TokenType's inferred type should be (label: string, conf: { binop: string }) based on usage in x.

Actual behavior:
TokenType's inferred type is (label: any, conf: { binop: null }), based only on usage inside TokenType itself.

I don't think infer-from-usage supports any JS assignment patterns; this is just the first one I ran into.

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

Beginne mit dem acorn-Beispiel im Issue und verfolge die infer-from-usage-Verarbeitung für den TokenType-Konstruktor und die x-Funktion. Überprüfe, wie Zuweisungsmuster und Referenzen verwandter Tokens gesammelt werden; abgeschlossen ist die Aufgabe, wenn der inferierte Typ label: string und conf: { binop: string } statt any und null enthält.

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
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

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