microsoft / microsoft/TypeScript

Don't offer to change spelling to not-yet-defined variables

Offen
#23,454 2 Kommentare 1 Reaktion 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.4k
Ø Merge
1 T. 19 Std.
Gemergte PRs (30 T.)
117

Beschreibung

TypeScript Version: 2.8.1

Search Terms: suggest change spelling auto fix import

Code

// This import is missing, but can be auto-suggested
import { /* SomeClass */ } from "./SomeClass";

const bindContainer = (container) => {
    const someClass = container.get(SomeClass);
};

Expected behavior:

When I ask for auto-fixes on the SomeClass, it should only suggest to add it to the import declaration.

Actual behavior:

Two suggestions are given:

  • Change spelling to 'someClass'
  • Add 'SomeClass' to existing import declaration from "./SomeClass"

We can statically reject the rename suggestion here because this is in the initializer of the variable.

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, das TypeScript-2.8.1-Beispiel zu reproduzieren und Auto-Fixes für SomeClass anzufordern. Verfolge die Logik, die Rechtschreibänderungen und den Vorschlag für einen vorhandenen Import anbietet, und überprüfe anschließend, dass der Rechtschreibvorschlag ausgelassen wird, während der Importvorschlag erhalten bleibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
typescript
Bereich
compilers
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
42/100

Neue Issues direkt in Ihr Postfach

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