Prefer type correct import for add missing import for quick fixes

Ouverte
#42,615 6 commentaires 1 réaction 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
3/5
Temps estimé
1-2 jours
Accessibilité débutants
42/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
À l'abandon
Stack technique
typescript
Domaine
developer-experience

Piste de recherche

Reproduisez la correction rapide dans index.ts à l’aide des exemples a1.ts et a2.ts, puis suivez le classement des corrections rapides d’auto-importation à partir de la référence abc dans foo(abc). Vérifiez que le candidat qui exporte la valeur compatible avec le type est classé en premier, et ajoutez ou mettez à jour la couverture pour ce cas si les tests de correction rapide pertinents sont trouvés.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

Awaiting More Feedback Suggestion

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
  1. Trigger the quick fix on abc in foo(abc)
🙁 Actual behavior

The import quick fixes are returned in file order:

Screen Shot 2021-02-02 at 12 45 53 PM

Accepting the first suggestion results in invalid code:

Screen Shot 2021-02-02 at 12 46 42 PM
🙂 Expected behavior

In this case, we should sort the import for a2 before a1 since it has the correct type

Langage dominant
Go
Étoiles
111k
Forks
14.4k
Merge moyen
1 j 19 h
PR mergées (30 j)
117

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de microsoft/TypeScript

Toutes les issues de microsoft/TypeScript

Issues similaires

Plus d'issues Go

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.