Microsoft/TypeScript

Do not rename imports from ambient modules

Open

#8 227 ouverte le 20 avr. 2016

Voir sur GitHub
 (6 commentaires) (3 réactions) (0 assignés)TypeScript (6 726 forks)batch import
Effort: ModerateHelp 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

Issue explained by @bbgone in https://github.com/Microsoft/TypeScript/issues/8118#issuecomment-212218038

Renaming an import alias for a module coming from a .d.ts file, renames all instances. this is obviously wrong.

Options, 1. error (obviously not helpful), 2. only rename local symbol (better, but leaves the code in an invalid state), 3. rename local import and add as oldname clause to the import declaration as needed (looks like the best solution).

This is similar to the issue related to https://github.com/Microsoft/TypeScript/issues/7458, except that this issue adds renaming the import alias.

Guide contributeur

Direction de recherche
Examinez le fournisseur de renommage dans le service linguistique TypeScript. Concentrez vous sur la façon dont les importations provenant de modules ambiants (.d.ts) sont traitées lors du renommage. Consultez les tests existants pour le renommage (par exemple, tests/cases/fourslash/renameImports*) et le code source du module de renommage. Assurez vous que le renommage d'un alias d'importation à partir d'un module ambiant ne renomme que le symbole local et ajoute une clause 'as' si nécessaire.
Stack technique
typescript
Domaine
compilers
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
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.
60