microsoft / microsoft/TypeScript
Don't offer to change spelling to not-yet-defined variables
Nessuno ha ancora preso questa issue.
- Lingua principale
- Go
- Stelle
- 111k
- Fork
- 14.4k
- Merge medio
- 1g 19h
- PR unite (30g)
- 117
Descrizione
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.
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia riproducendo l’esempio di TypeScript 2.8.1 e richiedendo correzioni automatiche su SomeClass. Segui la logica che offre modifiche ortografiche e il suggerimento per un import esistente, quindi verifica che il suggerimento ortografico venga omesso mentre quello dell’import rimanga.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- typescript
- Ambito
- compilers
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 42/100