microsoft / microsoft/TypeScript

Implement interface should not add optional properties

Offen
#22,770 12 Kommentare 5 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

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

Beschreibung

From https://github.com/Microsoft/vscode/issues/46286

TypeScript Version: 2.8.0-dev.20180320

Search Terms:

  • implement interface
  • quick fix
  • code action
  • optional property

Code

interface IFoo {
    x?: number
    y: number
}

class Foo implements IFoo {

}

Trigger implement interface on Foo

Expected behavior:
Only required properties required:

interface IFoo {
    x?: number
    y: number
}

class Foo implements IFoo {
    y: number;
}

Actual behavior:
Both required and optional properties added:

interface IFoo {
    x?: number
    y: number
}

class Foo implements IFoo {
    x?: number;
    y: number;
}

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 bei der TypeScript-Sprachdienst-Codeaktion, die durch „implement interface“ aufgerufen wird, und reproduziere das bereitgestellte IFoo/Foo-Beispiel. Prüfe die generierten Implementierungselemente und verifiziere anschließend, dass nur die erforderliche Eigenschaft y hinzugefügt wird, während das optionale x weggelassen wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
typescript
Bereich
tooling
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 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.