Microsoft/TypeScript

[Rename symbol command] Remove unnecessary named import

Open

#56.497 geöffnet am 21. Nov. 2023

Auf GitHub ansehen
 (2 Kommentare) (1 Reaktion) (0 zugewiesene Personen)TypeScript (6.726 Forks)batch import
Effort: CasualExperience EnhancementHelp WantedSuggestion

Repository-Metriken

Stars
 (48.455 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 6T 17h) (9 gemergte PRs in 30 T)

Beschreibung

  • VS Code Version: 1.84.2 (1a5daa3a0231a0fbba4f14db7ec463cf99d7768e)
  • OS Version: macOS Sonoma 14.1.1

Steps to Reproduce:

  1. Create 2 JS files:
// utils.js
export const foo = () => console.log('Hello world!');

// index.js
import { foo as bar } from "./utils.js";

bar();
  1. Put your cursor in the word "foo" in utils.js and use the "Rename Symbol" command (F2 by default) to rename the function to bar.
  2. Observe the import in index.js has changed to import { bar as bar } from "./utils.js". Since the names import is no longer necessary, the command should instead change the import to import { bar } from "./utils.js".

Contributor Guide

Research-Richtung
Untersuchen Sie die Rename Symbol Logik im TypeScript Sprachdienst, um Fälle zu behandeln, in denen der umbenannte Name mit dem Importnamen übereinstimmt und der unnötige Alias entfernt wird.
Tech Stack
typescript
Domain
compilersdeveloper experience
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TypeScriptGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70