microsoft / microsoft/TypeScript
Don't offer to change spelling to not-yet-defined variables
Dieses Issue hat noch niemand übernommen.
- 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
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Ö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