-
-
Notifications
You must be signed in to change notification settings - Fork 69
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
feat: better cache support via hashobject for example #158
Comments
I've made a reproduction of the issue here https://github.com/thatsmydoing/eslint-import-resolver-typescript-158 On my machine, the v2.7.1 branch takes about 2.30s
whereas master uses 3.4.0 takes around 27s to run
|
@thatsmydoing Thanks for reproduction, that would make fixing easier. Or you can help to contribute by yourself! |
I made an attempt at #166 but I don't think it's possible to get to the previous level of performance with enhanced resolve. |
Thanks, that's why I notice to use |
@thatsmydoing Can you help to try |
For the sample project and our project, the options object wasn't changing so it was perfectly cached even without hashing it. So there was no additional performance improvement. It's really just |
@thatsmydoing Thanks for verifying, maybe #165 will help. |
No description provided.
The text was updated successfully, but these errors were encountered: