consolidate.js icon indicating copy to clipboard operation
consolidate.js copied to clipboard

[question] ommitting loaders when using custom nunjucks env

Open frankli0324 opened this issue 1 year ago • 1 comments

just being curious, I found that consolidate reads the template and handles the string to nunjucks renderString:

https://github.com/tj/consolidate.js/blob/ec2e1ae511d37a69716022311a4ca29785f62143/lib/consolidate.js#L1329

https://github.com/tj/consolidate.js/blob/ec2e1ae511d37a69716022311a4ca29785f62143/lib/consolidate.js#L182

so may I consider it safe not to specify the loaders when creating a custom nunjucks environment with options.nunjucksEnv, which defaults to using a filesystem/webloader loader with app/views/views as it's root path? in other words, will this behavior change?

also, I didn't see any sign of active maintainence for this repository, is this library deserted?

frankli0324 avatar Oct 14 '22 13:10 frankli0324

We have forked this repository for maintenance and released it under @ladjs/consolidate, see https://github.com/ladjs/consolidate.js. We have merged PR's and updated it for email-templates. Please click the "Watch" button to get notified of all releases at https://github.com/ladjs/consolidate.js. Thank you 🙏

Screen Shot 2023-06-08 at 3 05 12 PM

PR welcome at the new repo once new release is published today!

titanism avatar Jun 08 '23 20:06 titanism