-
-
Notifications
You must be signed in to change notification settings - Fork 534
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
Restore cache functionality (default to off) #951
Comments
Discussion about how to re-add caching is over here: #908 That thread has my thoughts on how to implement disk caching. For my use-cases, I'm focused on using it with transpile-only mode, since my projects perform typechecking as a separate step. It should be safe to enable automatically with transpile-only when we're not using transformers. For your case it would still need to be behind an opt-in flag, as you say. Even then, maintainers might be against it if it causes an influx of false bug reports. But I can't speak for anyone else. |
SGTM behind a flag 👍 |
Do you know when you'll be able to get to this? My team would like to use ts-node but the performance without caching is a huge blocker. |
@ecraig12345 not sure, pull requests are welcome. In the meantime you can use typescript-cached-transpile, linked over here: #908 Turning on |
Closing since I doubt we'll get a PR for this, and If any sort of caching does get implemented, it'll probably be related to TS's incremental builds. |
#701 removed caching and I find performance of using ts-node unbelievably slow in any version beyond v7.0.1. The commit which removes caching behavior is in response to #672 (Cache doesn't take modified dependencies into account). I use ts-node in combination with
TS_NODE_FILES=true
and have never hit this problem nor was I able to reproduce the issue.It seems that the maintainer is willing to accept a PR to add caching behavior in #920. Before I submit a PR, I'd like to confirm if it will be acceptable to restore the previous behavior, but have caching disabled by default and update the README to state that this is discouraged and link th #672 as to the reason why?
The text was updated successfully, but these errors were encountered: