Andrew Branch

Is program time being reported as part of parse time? If the numbers are to be believed, looks like the impact might actually be visible. We are talking about 1/100th...

> looks like the impact might actually be visible It was probably just a coincidence in the noise.

I’ve only just glanced at the implementation, but re: naming—in #51171 I introduce `allowImportingTsExtensions`, and I think I’ve managed to convince myself that its functionality really shouldn’t be combined with...

Also: don’t you need corresponding changes in moduleSpecifiers.ts and path completions? E.g., what happens in declaration emit when you have: ```ts // foo.d.html.ts export declare class CustomHtmlRepresentationThing {} export declare...

> As an added bonus, .ts file imports now resolve during module resolution, but we still issue retain an error on the import in the checker, for our usual "ts...

Nothing should change here because this is mostly adding code that no existing projects are able to touch, but: @typescript-bot perf test this @typescript-bot run top100

Closed by #51435

5.2* It’s @rbuckton’s PR to merge whenever he’s ready.

I think I’d rather not merge it two days before the stable release. I’ll merge once main is open to 5.4-bound code if Ron doesn’t first.

Are we about to publish 5.2? I thought we were about to publish 5.3. Anyway, if we’ve already branched, we can go 😄