Prefer type correct import for add missing import for quick fixes

Offen
#42,615 6 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Anfängerfreundlichkeit
42/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Veraltet
Tech-Stack
typescript
Bereich
developer-experience

Rechercherichtung

Reproduziere die Schnellkorrektur in index.ts anhand der Beispiele a1.ts und a2.ts und verfolge dann die Rangfolge der Auto-Import-Schnellkorrekturen anhand der abc-Referenz in foo(abc). Überprüfe, dass der Kandidat, der den typkompatiblen Wert exportiert, an erster Stelle steht, und füge für diesen Fall Tests hinzu oder aktualisiere die Abdeckung, falls die relevanten Schnellkorrekturtests gefunden werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

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

Vorherrschende Sprache
Go
Sterne
111k
Forks
14.4k
Ø Merge
1 T. 19 Std.
Gemergte PRs (30 T.)
117

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus microsoft/TypeScript

Alle Issues in microsoft/TypeScript

Ähnliche Issues

Weitere Issues zu Go

Neue Issues direkt in Ihr Postfach

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