microsoft / microsoft/TypeScript
Prefer type correct import for add missing import for quick fixes
Nessuno ha ancora preso questa issue.
- Lingua principale
- Go
- Stelle
- 111k
- Fork
- 14.4k
- Merge medio
- 1g 19h
- PR unite (30g)
- 117
Descrizione
Bug Report
From https://github.com/microsoft/vscode/issues/115370
🔎 Search Terms
- Quick fix
- auto import
🕗 Version & Regression Information
Tested version: 4.2.0-dev.20210201
⏯ Playground Link
N/A
💻 Code
For the project:
// a1.ts
export const abc = 123
// a2.ts
export const abc = '123'
// index.ts
function foo(x: string) { }
foo(abc); // trigger quick fix here
- Trigger the quick fix on
abcinfoo(abc)
🙁 Actual behavior
The import quick fixes are returned in file order:
Accepting the first suggestion results in invalid code:
🙂 Expected behavior
In this case, we should sort the import for a2 before a1 since it has the correct type
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
Riproduci la correzione rapida in index.ts usando gli esempi a1.ts e a2.ts, quindi traccia l'ordine di classificazione delle correzioni rapide di importazione automatica a partire dal riferimento abc in foo(abc). Verifica che venga messo al primo posto il candidato che esporta il valore compatibile con il tipo e aggiungi o aggiorna la copertura per questo caso se vengono trovati i test pertinenti delle correzioni rapide.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- typescript
- Ambito
- developer-experience
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 42/100