Drop callback support in v2

wesleytodd commented 5 years ago

I am starting work on what will hopefully be v2 of this module. One of the breaking changes I was thinking of making is to drop the callback interface for migration up/down functions and other exposed api's. What do people think of this? Acceptable?

The benefits are:

reduced code and documentation complexity
promote use of async/await which is a better paradigm
has full support in active node versions

The cons:

Callbacks have a long history, users might expect them
The feature exists today, we would probably want to deprecate them in a 1.x release soon if we plan to break it in 2.x

Thanks in advance for your thoughts.

LinusU commented 5 years ago

I'm 100% for this 👍

palmerabollo commented 3 years ago

@wesleytodd I'd be happy to try the v2, even if it is still "beta". Is it available somewhere? Thank you.

wesleytodd commented 3 years ago

Hey @palmerabollo, I never finished it :(

While I have a bunch of work in https://github.com/migratejs I don't think it got back to a working state. I do have some projects at work where I plan to use it, and thus finish that but I am not quite there yet.

tj / node-migrate

Drop callback support in v2 #160