Humdum, I was wondering why stuff doesn't work when deployed and then realised that I forgot to add a require
for honeysql-postgres.format
in order to get multimethods registered. Not the first time I do this mistake, maybe seldom enough that I always forget about it :D
if v2 has more stuff baked in, this won't be a problem I guess
Yeah, I'm looking forward to removing that library too
the inclusion of the format
always threw me
@viesti My current plan is to add everything in the nilenso lib directly into V2 core and just treat a combination of ANSI/PostgreSQL syntax as the default, overriding it for MySQL as needed. And there will be no multimethod stuff: just register-*!
functions to add new SQL clause syntax, new infix operators, and new "special syntax". But overall I'm hoping to get enough into the core that no one needs to register new stuff (or, if they do, they can just let me know and I'll add it to the core anyway).