TJ
TJ
yup! for now at least, ideally separate pull-requests so they're not getting mixed up
yes includes should be relative to the file using the directive, if not then that's a bug
I found it a little weird to use `sqlx.NamedExec()` (just discovered that) in place of the DB/TX satisfying an interface that has `NamedExec()` and the others. Not hard to add...
nope, not yet at least
@dazoe you mean express 3
yeah I'll have to come up with something for ejs, bring partial() back maybe but that doesn't help for layouts
ejs has includes now but yup i'll put some effort into the extends part soonish
the include I added is at compile time like jade so the sync IO doesn't matter, but it would be nice to still have a function and cache that sync...
yup, doing any IO at all is unnecessary in most cases. I agree though a simple function is nicer than some new concept. What I used to do with partials...
there's nothing improper about include, but yeah a partial() / layout would be nice