-
Notifications
You must be signed in to change notification settings - Fork 38.8k
Closed
Labels
debtCode quality issuesCode quality issuesinsiders-releasedPatch has been released in VS Code InsidersPatch has been released in VS Code Insidersperfperf-startupterminalGeneral terminal issues that don't fall under another labelGeneral terminal issues that don't fall under another label
Milestone
Description
I see no reason why node-pty needs to be loaded sync, as I think it is only used when an integrated terminal is opened. Loading it via await import('node-pty') could save some ms on startup:
//cc @jrieken
Reactions are currently unavailable
Metadata
Metadata
Labels
debtCode quality issuesCode quality issuesinsiders-releasedPatch has been released in VS Code InsidersPatch has been released in VS Code Insidersperfperf-startupterminalGeneral terminal issues that don't fall under another labelGeneral terminal issues that don't fall under another label
