search

nilenso / honeysql-postgres

PostgreSQL extension for honeysql
http://nilenso.com
Eclipse Public License 1.0
193 stars 28 forks source link

Require the format namespace in the helper #27

Closed arichiardi closed 3 years ago

arichiardi commented 5 years ago

Hi there!

I found myself including two namespaces, one for the helpers and the other one for the format and questioned "what if the helper required format so that I don't have to remember?"

There must be a reason why we don't do it though so I am opening an issue in order to ask 😉

ukriish commented 5 years ago

@arichiardi I've kept it this way to keep this in accordance with how honeysql is written. I didn't want to change much to keep it familiar.

Actually, I think you have a point. That should work. 🤔

kitallis commented 3 years ago

This was sorted in the 0.3.104 release.

cc @arichiardi