-
Notifications
You must be signed in to change notification settings - Fork 6
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
does not appear persistent #5
Comments
@stefanpenner Do you mean using |
@kratiahuja ya, its confusing (most likely the author of persistent-filter itself fault) TL;DR persistent-filter provides a more efficient and stable filter subclass, but the persistent part is opt-in. This is because cache keys are hard, and require extra thought. https://github.com/stefanpenner/broccoli-persistent-filter#persistent-cache may help, let me know if it does not I would love some input on how to improve the docs. |
Aah ok, this makes sense now. I didn't realize it was an opt-in. I'll read through the docs. I'll look into this sometime this week :) . Thank you @stefanpenner ! |
The TS linter appears sufficiently costly, that enabling the persistent mode would be a nice startup improvement.
The text was updated successfully, but these errors were encountered: