Rik Smale

A quick workaround is to remove this line locally; https://github.com/sequelize/cli/blob/a2526ed0090bcd55eaeeb0179f79ea6e09cd1a84/src/core/migrator.js#L132 We officially do not support memsql/SingleStore btw. What dialect did you use in the sequelize config?

@ephys I think you can answer what the plans are for sequelize v7 (either already in the beta or in a later minor release) For context; the functionality from sequelize-typescript...

In sequelize v6 there are some features in sequelize-typescript that makes working with TypeScript easier. But it doesn't replace sequelize. In sequelize v7 these features are part of sequelize itself...

Heads up for these and your other open PRs; we've moved to prettier in #17101 which causes a lot of merge conflicts. When resolving you can just accept your changes...

@lohart13 @ephys just so that we are on the same page; is this order of when we want to review/merge PRs correct? 1. #17157 and #17158 (these are interchangeable from...

@ephys even though it's in draft; could you take a look at the test cases that we do? As mentioned in the TODOs above, there is still work to be...

I'll mark it as ready for review since the bulk of the work is done. It's not ready for merging yet, since I want to know if there's any tests...

> Ok, I've added the Temporal support now however the test will most likely fail due to not all of the Temporal features being available in the lts versions of...

> Shall I close the PR for now and reopen it when it's ready so the automated checks also stop running? No need, it's fine. We don't pay for the...

@AjaniBilby took a while, but we've merged https://github.com/sequelize/sequelize/pull/16283 so you should be able to get passing tests on Windows