better error reporting when parsing queries about hugsql HOT 4 CLOSED

What, you don't like a good Clojure NullPointerException hunt? Ha, just kidding! I've actually run into this myself transitioning from Yesql. I'll add in a validation check for this and get it into the next release.

Thanks for the report!

from hugsql.

fantastic! 👍

from hugsql.

This is released in 0.4.3.

The strategy at this point is to give a little bit of context about which header keys it did find and to dump the sql template. Since the parser is generically picking up HugSQL headers (it doesn't expect :name or its variants, just :some-key), it doesn't catch this during the parse pass, so it checks for this right before the def pass. A later enhancement might set the reader line and column meta data for each statement, but the current fix should at least keep you from the dreaded NullPointerException.

from hugsql.

great, I think we can close this one unless you want to keep it here for the later enhancement part :)

from hugsql.