Alyx

Results 453 comments of Alyx

Somewhat, but definitely not with the same ergonomics The best we have in Sequelize 6 is a mix of `literal` and `sequelize.escape`, but there is no good alternative to `sql.attribute`,...

It's `sequelize.escape`, not `queryInterface.escape`

Woops, misclick from using github on mobile sequelize.escape: https://github.com/sequelize/sequelize/blob/a3213f053bcc4534073e56def64f22392edaf28c/src/sequelize.js#L704 https://github.com/sequelize/sequelize/blob/a3213f053bcc4534073e56def64f22392edaf28c/src/sequelize.d.ts#L1301

`escape` is an instance method, not a static one

So I don't forget: This PR is blocked, we need to open another PR to clean up `queryInterface.describeTable` to ensure it returns consistent results accros all dialects

Yep! I already introduced the internal query generator files & other similar refactors in a recent PR, the `canHaveDefaultValue` logic could be moved too, as well as the changes to...

(ignore notification, was demoing something, sorry)

Everything described in here is implemented now

@mike-usa hey :) Is this something you still want to work on?