TJ

Results 725 comments of TJ

I think one Config arg is cleaner personally, easily enough to document a field as required or panic() if something is missing. I think it's also more future-proof because you...

I suppose it's still breaking API if you add a required field haha, but oh well, beats (a, b, c, d, e, f, finallyConfig) I think

its a symlink to lib/redis-client.js so you can clone (or curl the tarball) into ~/.node_libraries directly

i guess so, kinda defeats the purpose of a template IMO but sure why not it would be simple to add

jade has similar to disable the debugging stuff. then we could just have `ejs.compile(str, options).toString()` to get the fn string

option sounds good to me

probably best to leave it enabled by default to BC

pull-requests are welcome :D

i like the intent here but it might take me a while to review since you rewrote a pretty large portion of it

yeah understandable, we should have created an AST like Jade but the scope of EJS kinda changed with these features. im implementing includes right now the way that I would...