Microsoft/TypeScript

Give a better error message when an import isn't found but a namespace of the same name exists

Aperta

#10.464 aperta il 21 ago 2016

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)TypeScript (13.395 fork)batch import
BugDomain: Error MessagesEffort: ModerateHelp Wanted

Metriche repository

Star
 (108.860 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

See #10463.

foo.d.ts

declare module foo {
    // ...
}

bar.ts

import foo = require("foo");

"foo" won't be found, but if the compiler can resolve an identifier named foo to a namespace that uses the declare module syntax, we should probably give an error like

Cannot find module 'foo'.
  A legacy-style namespace with the same name was found.
    Try changing the declaration to use quotes (e.g. 'declare module "foo"').

Guida contributor

Direzione di ricerca
Ispeziona il codice di risoluzione dei moduli nel compilatore TypeScript (probabilmente in src/compiler/moduleResolver.ts) e trova dove viene emesso l'errore 'Cannot find module'. Quindi aggiungi la logica per verificare se esiste un namespace con lo stesso nome (tramite la sintassi declare module) e suggerisci la correzione nel messaggio di errore.
Tech stack
typescript
Dominio
compilers
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75