microsoft / microsoft/TypeScript
If autoimporting import that is commented out - uncomment it instead of adding new one
Personne n'a encore pris cette issue.
- Langage dominant
- Go
- Étoiles
- 111k
- Forks
- 14.3k
- Merge moyen
- 2 j 4 h
- PR mergées (30 j)
- 132
Description
Search Terms
autoimport commented uncomment import
Suggestion
Consider we have such imports already in some file:
import React, {
useState,
// useEffect,
} from 'react';
Then - if you write useEffect - TS will suggest auto importing it - which is cool. If you choose to auto import it - your import statement will look something like:
import React, {
useState,
// useEffect,
useEffect,
} from 'react';
Which is not a big deal, but leads to some kind of duplication.
Desired result would be:
import React, {
useState,
useEffect,
} from 'react';
Which is just removing the comment next to the import I want to use.
Checklist
My suggestion meets these guidelines:
- This wouldn't be a breaking change in existing TypeScript/JavaScript code
- This wouldn't change the runtime behavior of existing JavaScript code
- This could be implemented without emitting different JS based on the types of the expressions
- This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
- This feature would agree with the rest of TypeScript's Design Goals.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par le traitement des auto-imports du language-service de TypeScript et trouvez les tests qui couvrent les imports avec des imports nommés commentés. Ajoutez un cas de régression pour l’import de style React présenté et vérifiez que la sélection de l’auto-import supprime le commentaire au lieu d’ajouter un import en double.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- typescript
- Domaine
- tooling
- Type d'issue
- Fonctionnalité
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100