Skip to content
This repository has been archived by the owner on Jun 15, 2023. It is now read-only.

Introduce Var newtype #41

Open
garyb opened this issue Jan 4, 2018 · 0 comments
Open

Introduce Var newtype #41

garyb opened this issue Jan 4, 2018 · 0 comments

Comments

@garyb
Copy link
Member

garyb commented Jan 4, 2018

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 😕)

@garyb garyb changed the title Introduce SqlIdent newtype Introduce Var newtype Jan 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

No branches or pull requests

1 participant