microsoft / microsoft/TypeScript
Add TS server command for removing all unused code in a file
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
- code action
- source action
- getCombinedCodeFix
Feature request
Add a new TS Server command that removes all unused code in a file. This should only perform 'safe' removals, such as:
- Unreachable code
- Unused imports
- Unused declarations
We should likely not remove unused parameters. Not sure about removing unused variable since removing them can also have side effects
Use case
VS Code allows users to configure code actions that are run on save. One of the supported code actions is source.removeUnused which attempts to remove unused code in a file
On the VS Code side, we can't implement this very well since unused code often ends up overlapping with other unused code and we don't have much insight into what is "safe" to remove and what is not
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 lire les commandes TS Server existantes et le chemin getCombinedCodeFix mentionné dans l’issue, puis comparez la manière dont VS Code invoque source.removeUnused. Le travail est terminé lorsqu’une commande supprime uniquement le code inaccessible sans risque, les imports inutilisés et les déclarations inutilisées, tout en laissant les paramètres et les variables susceptibles d’avoir des effets de bord inchangés.
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é
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100