microsoft / microsoft/TypeScript

Quick fix to declare overridden method on base class

Offen
#45,172 2 Kommentare 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

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

Beschreibung

From https://github.com/microsoft/vscode/issues/129130#issuecomment-885409056

Suggestion

Add a new quick fix on a subclass that tried to override a non-existent method to add that method to the base class.

For example, starting with:

class Base {

}

class Sub extends Base {
	override foo() {}
}

The quick fix would generate:

class Base {
	foo() {
		throw new Error("Not implemented");
	}
}

class Sub extends Base {
	override foo() {}
}

🔍 Search Terms

  • Override
  • quick fix
  • code action

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 damit, die Verarbeitung von Quick-Fixes oder Code-Aktionen des TypeScript-Sprachdienstes für das Überschreiben einer nicht vorhandenen Methode zu lokalisieren. Verfolge, wie die Diagnose erzeugt wird, und prüfe anschließend die vorhandenen Quick-Fix-Tests, bevor du eine Abdeckung für das Beispiel mit Base und Sub hinzufügst. Als abgeschlossen gilt die Aufgabe, wenn der Quick-Fix die Basismethode mit dem gezeigten nicht implementierten Methodenrumpf erzeugt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
typescript
Bereich
compilers
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.