gajus / slonik

A Node.js PostgreSQL client with runtime and build time type safety, and composable SQL.
Other
4.58k stars 139 forks source link

Deprecate sql.rawList in favour of sql.expressionList #91

Closed gajus closed 5 years ago

gajus commented 5 years ago

sql.rawList misleading implies use of sql.raw as a requirement.

Same goes for sql.valueList, sql.tupleList, sql.identifierList and sql.booleanExpression.

All of them can be replaced with a generic sql.expressionList that works similar to .join().

gajus commented 5 years ago

And add an option to specify glue expression.

gajus commented 5 years ago

This has been already done.

rakibtg commented 4 years ago

Is expressionList is still valid?

gajus commented 4 years ago

No. Just use sql.join.