Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use transpile only version of ts-node for better performance during runtime load #69

Closed
wants to merge 1 commit into from

Conversation

a7ul
Copy link

@a7ul a7ul commented Jun 10, 2020

Hi

when using interpret with typescript files and ts-node, there is significant initial load time.
To improve this, ts-node has transpile-only option which is fast and production ready.
This PR add transpile only option to ts-node register.

For more details: TypeStrong/ts-node#104 (comment)

@phated
Copy link
Member

phated commented Jun 10, 2020

Since this is using the same module, this would be a breaking change.

The goal of using Surcrase in #66 was performance. Maybe you want to switch?

@phated
Copy link
Member

phated commented Jun 10, 2020

cc @flipsasser and @blakeembrey ☝️

@blakeembrey
Copy link
Contributor

This probably makes sense, I'm not sure if people do or do not expect their code to be type checked when it's loaded with interpret. Is there any philosophical stance in interpret? The file was added in V6 in 2018 so has fairly large adoption.

@flipsasser
Copy link
Contributor

Most of this is above my pay grade - but I recommend giving Sucrase a look if you're looking for something extremely fast, @a7ul

@a7ul
Copy link
Author

a7ul commented Jun 23, 2020

Yep, I have switched over to sucrase for now. Thanks

@a7ul a7ul closed this Jun 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants