search

slamdata / purescript-quasar

Quasar API library for PureScript
Apache License 2.0
1 stars 7 forks source link

Use SqlSquared.Sql instead of String #84

Closed safareli closed 6 years ago

safareli commented 6 years ago

Note: this is braking change

garyb commented 6 years ago

@safareli looks like one of the tests is failing due to invalid sql (there's a partial pattern error), so maybe Maxim's idea isn't so bad? :wink:

I'm fine with it either way for test purposes though, parsed strings or AST construction.

garyb commented 6 years ago

Looks good :shipit: