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

Dust.js cached partials (in development env) #84

Closed
me-ascii opened this issue Jan 7, 2013 · 5 comments
Closed

Dust.js cached partials (in development env) #84

me-ascii opened this issue Jan 7, 2013 · 5 comments

Comments

@me-ascii
Copy link

me-ascii commented Jan 7, 2013

Based on: #33

Have to clean dust.cache = {} each time render.
Please revert the auto-cleaning.

@ForbesLindesay
Copy link
Contributor

I'm not clear, why should the auto-clearing of the cache be removed?

@me-ascii
Copy link
Author

me-ascii commented Jan 8, 2013

If use partials you have to restart the server after each partial-templates update.
Because dust stores compiled partials, even if the { cache: false } flag is set
By default dust keeps all compiled files in dust.cache

@ForbesLindesay
Copy link
Contributor

Ah, I get it, the change was reverted, and you want the behavior that that pull request introduced back?

This got lost by accident in e0cabb1

@ForbesLindesay
Copy link
Contributor

This is a dupe of #76 but that needs re-basing anyway. Once #85 is merged I'll submit a P.R. for dust.

P.S. I'll also make the same change in transformers (my own slightly more feature rich version of consolidate.js) right away.

@doowb
Copy link
Collaborator

doowb commented Sep 17, 2017

Closing this since the dust engine cache is cleared now before rendering a file.

@doowb doowb closed this as completed Sep 17, 2017
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

No branches or pull requests

3 participants