brunch / nunjucks-brunch

Adds nunjucks support to Brunch
8 stars 6 forks source link

reuse Environment to prevent memory leak warnings #3

Closed Ramblurr closed 9 years ago

Ramblurr commented 9 years ago

This fixes warnings like:

(node) warning: possible EventEmitter memory leak detected. 11 listeners added. Use emitter.setMaxListeners() to increase limit.

PxlBuzzard commented 9 years ago

Looks good, thank you for the fix.

Ramblurr commented 9 years ago

@PxlBuzzard any chance you could look at #2 ?

I think the plugin needs to be run on onCompile instead of compile, and build all the templates not just the changed ones (see how ilkosta/static-jade-brunch does it)