Microsoft/TypeScript

[Rename symbol command] Remove unnecessary named import

Open

#56 497 ouverte le 21 nov. 2023

Voir sur GitHub
 (2 commentaires) (1 réaction) (0 assignés)TypeScript (6 726 forks)batch import
Effort: CasualExperience EnhancementHelp WantedSuggestion

Métriques du dépôt

Stars
 (48 455 stars)
Métriques de merge PR
 (Merge moyen 6j 17h) (9 PRs mergées en 30 j)

Description

  • 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".

Guide contributeur

Direction de recherche
Examinez la logique de Renommage de Symbole dans le service de langage TypeScript pour gérer les cas où le nom renommé correspond au nom d'importation, supprimant ainsi l'alias inutile.
Stack technique
typescript
Domaine
compilersdeveloper experience
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
TypeScriptGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70