Microsoft/TypeScript

Do not rename imports from ambient modules

Open

#8227 aperta il 20 apr 2016

Vedi su GitHub
 (6 commenti) (3 reazioni) (0 assegnatari)TypeScript (6726 fork)batch import
Effort: ModerateHelp WantedSuggestion

Metriche repository

Star
 (48.455 star)
Metriche merge PR
 (Merge medio 6g 17h) (9 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Investiga il provider di rinominazione nel servizio linguistico di TypeScript. Concentrati su come vengono gestite le importazioni da moduli ambientali (.d.ts) durante la rinomina. Esamina i test esistenti per la rinomina (es. tests/cases/fourslash/renameImports*) e il codice sorgente del modulo di rinomina. Assicurati che rinominare un alias di importazione da un modulo ambientale rinomini solo il simbolo locale e aggiunga una clausola 'as' se necessario.
Tech stack
typescript
Dominio
compilers
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60