search

andywer / squid

🦑 Provides SQL tagged template strings and schema definition functions.
MIT License
148 stars 7 forks source link

uuid column type added #40

Closed AgentGoldPaw closed 2 years ago

AgentGoldPaw commented 3 years ago

I use the uuid column type in postgres. This change allows for the definition of a uuid column. Will update with corresponding change in postguard. https://github.com/andywer/postguard/pull/36

andywer commented 3 years ago

Just one little request: Since it's also JSON.parse() in JS and it's already Schema.JSON, let's make it Schema.UUID (all upper-case), too. Otherwise fine 👍

AgentGoldPaw commented 3 years ago

I will make the change a little later today. Currently not able to, but that makes sense.

AgentGoldPaw commented 3 years ago

renamed correctly, I am still somewhat new to typescript. But does the uuid column need to be added here? https://github.com/andywer/squid/blob/master/src/schema.ts#L44