Jake Bailey
Jake Bailey
Yeah, TS internally has no notion of a "set only property", see https://github.com/microsoft/TypeScript/issues/58112 and https://github.com/microsoft/TypeScript/issues/58119 (Void is definitely not the right answer, though)
The above test crashes even in TS 4.5 (and back even in 4.0 though with a slightly different error). So this most definitely is not a recent regression at all.
I think it may be sufficient to just put the namespace import before a named/default import, actually. If you're importing with names but twice, that's weird and probably pointless as...
@typescript-bot perf test this faster
Is this the winning PR or is #58593 still something to consider? Just checking before I review...
I don't think that it is guaranteed; I almost made the playground always set forceDtsEmit because it always needs to show it (playground used to crash on dts errors), though...
Isn't it public via `getEmitOutput` #34495? Or does that not matter?
#58364 is in, so is this safe to update and get in?
@typescript-bot test top200 @typescript-bot user test this @typescript-bot run dt @typescript-bot perf test this @typescript-bot pack this
@typescript-bot test it