slamdata / purescript-sql-squared

purescript Sql^2 ast, pretty printer etc
11 stars 6 forks source link

Introduce `Var` newtype #41

Open garyb opened 6 years ago

garyb commented 6 years ago

That would avoid stringly typed Vari - it would mean we have to use printIdent rather than accidentally printing the plain string anywhere, and would also allow us to use that type instead of VM.Var in SlamData (currently we use VM.Var in most places involving SQL & varmap vars but then unwrap to a plain string when generating SQL 😕)