microsoft / microsoft/TypeScript
Correct declaration emit of commonjs exports of namespace-nested classes
Offen
@sandersn arbeitet bereits daran.
Seit 10.9.2020.
Bug
Domain: Declaration Emit
- Vorherrschende Sprache
- Go
- Sterne
- 111k
- Forks
- 14.3k
- Ø Merge
- 2 T. 4 Std.
- Gemergte PRs (30 T.)
- 132
Beschreibung
Followup to #40228:
var NS = {}
NS.K =class {
values() {
return new NS.K()
}
}
exports.K = NS.K;
Expected behavior:
Should produce d.ts output:
declare namespace NS {
class _K {
values(): _K;
}
export {_K as K};
}
import _K = NS.K;
export {_K as K};
Actual behavior:
export var K: {
new (): {
values(): any;
};
};
- Inlines and destroys the namespace.
- Only exports a value, not a class.
- Doesn't serialise nested typed references to
Kproperly.
The declaration emitter just needs code to support this; right now it's hitting inaccurate fallback code.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Bewertung
Dieses Issue wurde noch nicht bewertet.