honeysql

Discussion of https://github.com/seancorfield/honeysql :slightly_smiling_face:
bja 2017-08-28T00:47:51.000087Z

@michaelblume as usual I'm super late to the party, but please see my comment here regarding #177 and #179 (https://github.com/jkk/honeysql/issues/177)

bja 2017-08-28T00:48:47.000008Z

while it's not strictly wrong to use HoneySQL's ToSQL, we already have a mechanism that is likely preferable in using honeysql.format/value to wrap custom types that are already implementing clojure.java.jdbc/ISQLValue

bja 2017-08-28T00:50:03.000022Z

that said, it's super weird and hard to find honeysql.format/value and that should probably move to the types namespace with the rest of the custom types