honeysql

Discussion of https://github.com/seancorfield/honeysql :slightly_smiling_face:
viesti 2020-09-30T10:15:30.115200Z

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

👍 1
viesti 2020-09-30T10:15:51.115600Z

if v2 has more stuff baked in, this won't be a problem I guess

dharrigan 2020-09-30T13:15:32.115900Z

Yeah, I'm looking forward to removing that library too

dharrigan 2020-09-30T13:15:41.116200Z

the inclusion of the format always threw me

seancorfield 2020-09-30T17:06:12.119200Z

@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).

👍 1